[3.1][3.2] [Dev] Page Scrolling - Прокрутка страницы

[Perfecthus]

Название расширения: Page Scrolling
Автор: Perfecthus
Описание расширения: Расширение добавляет стрелки прокрутки на страницы форума и в админпанель. Стрелки не будут видны гостям и поисковым ботам.
Скачать: https://github.com/PWG-Extension/pagescrolling
Язык: en, ru

Версия phpBB: >=3.1.5, 3.2.*

Позже хочу добавить админку. Писал плагин прокрутки для движка е107, но там совершенно другое API, с phphBB пока так сильно не знаком, поэтому админки пока нет.

Снимок.png

В планах добавить в админку следующие настройки:

admpanel.png

Установка: Распаковать в каталог: root/ext/pwg/pagescrolling/ и активировать расширение в админпанели Персонализация -> Управление расширениями

Язык: Русский, English
PS: Спасибо Anvar`у за подсказки.

[staiki]

Прикольные стрелки. Поменял свои на эти. А можно цвет немного ярьче? Плоховато синим. Можно коричневый или чёрным?

[Perfecthus]

Можно коричневый или чёрным?

Можно, Photoshop решит вопрос за несколько секунд

[staiki]

Можно, Photoshop решит вопрос за несколько секунд

Я понял. Пошел осваивать фотошоп..

[Perfecthus]

Можно просто найти изображения стрелок в интернете

[staiki]

Та ладно. И эти прикольные. Благодарю ещё раз. Во всяком случае аккуратнее чем были у меня и более "живые".

[Perfecthus]

А про Фотошоп я не шутил, цвет поменять дело нескольких секунд

[staiki]

А про Фотошоп я не шутил, цвет поменять дело нескольких секунд

Для меня винду переустановить 15 минут. Каждому своё.

[KEMnEP]

Так как я люблю минимализм. И невижу смысла в лишних запросов для сервера. Убрал имхо всё лишние не влияющие на функционал. Авторство исходное сохранено.

[Perfecthus]

Для меня винду переустановить 15 минут

А, что какие-то сложности с установкой винды могут быть

невижу смысла в лишних запросов для сервера

О каких запросах речь? Вроде итак ничего лишнего не было.

[staiki]

А, что какие-то сложности с установкой винды могут быть

Да перевидел я таких уже не одну тысячу, которые говорил что сложного ничего нет. А потом как оказалось не знают как и в биос зайти и что это такое. Так что не будем улыбки показывать. Я имел ввиду только то,что каждый занимается тем что знает и что ему по-душе.Мне фотошоп не интересен - если вы до сих пор не поняли.

Убрал имхо всё лишние не влияющие на функционал.

Благодарю. Не знаю что там убиралось но судя по описанию поста заменил не сомневаясь. Благодарю.

[Sheer]

Вроде итак ничего лишнего не было

Лишние файлы ext.php и index.html
listener.php

Код: Выделить всё

/**
* @ignore
*/
if (!defined('IN_PHPBB')) {
    exit;
}
 

Этот кусок кода линий.

Код: Выделить всё

class listener implements EventSubscriberInterface { 

Фигурная скобка должна быть на новой строке

Код: Выделить всё

static public function getSubscribedEvents() { 

Аналогично.

Код: Выделить всё

			'core.user_setup'	=>	'load_language_on_setup',			

Три пустых табулятора в конце строки

Код: Выделить всё

public function load_language_on_setup($event) { 

Фигурная скобка должна быть на новой строке
После закрывающей фигурной скобки лишний табулятор
PageScrolling.php
Строки 21 и 25 - Фигурная скобка должна быть на новой строке
Строка 32 - пустая
Закрывающий тег ?> ненужен

KEMnEP, overall_footer_after.html

Код: Выделить всё

<div class="runup" title="Прокрутить страницу вверх" id="ToTop"></div><div class="rundown" title="Прокрутить страницу вниз" id="ToBottom"></div>

Это очень нехорошо, следует все же использовать языковые переменные.
overall_header_head_append.html

Код: Выделить всё

<link href="{BOARD_URL}ext/Predator/PageScrolling/styles/all/theme/PageScrolling.css" rel="stylesheet">

Совсем плохо. У автора сделано правильно. А если уж применять подобный метод, то вместо {BOARD_URL} нужно использовать {ROOT_PATH} и добавлять assets_version={T_ASSETS_VERSION}

[KEMnEP]

Sheer, С точки зрения сферического вакуума, Вы абсолютно правы, как и любая паническая программа валидации (работающая в том же вакууме и созданная людьми неглупыми, но людьми не могущими всё предусмотреть) показывающая обсолютно правильные в большинстве случаев вещи, но есть и исключения. Ибо слава богу мы в вакууме этом не живём, иначе бы задохнулись.
И так аргументирую мои правки, сразу скажу на истину последней инстанции я не претендовал и не претендую. Лишь показал как сделал у себя, может кто то с таким же технологическим складом ума примет те же решения, и с экономит время, любой другой может скачать оригинал, я его не замещаю.
Дано, расширение используемое на ВСЕХ страницах форума кроме админки, и ВСЕМИ пользователями, ботами и гостями. Посему его минимальная ресурсоемкость крайне важна.
Начнём с простого.

Это очень нехорошо, следует все же использовать языковые переменные.

Отсутствие титле, или вообще их присутствие не каждый пользователь у стрелок и заметит, мобильные устройства сразу идут стороной ибо не кажут их, там подвёл палец и перешёл, пользователи стационарных компов тоже не будут долго висеть мышкой читая что там, поэтому эти переменные в данном расширении в подавляющем большинстве случаев (сноска на статистическую погрешность запрещающию говорить 0) не затребованы, и лишь попусту тратят ресурсы серверной части. По идеи эти титле вообще можно стереть, и лишь параноидальная программа валидации заметит их отсутствие (и то заметит ли?). Я писал их для русского форума. поэтому у меня на русском. Но их вообще отсутствие для меня не проблема, или английский язык в них.
Посложней.

Совсем плохо. У автора сделано правильно. А если уж применять подобный метод, то вместо {BOARD_URL} нужно использовать {ROOT_PATH} и добавлять assets_version={T_ASSETS_VERSION}

{BOARD_URL} Стандартная переменная форума и наверняка где то по логике должна кешироваться не затрагивая расширение (не проверял) и из за этого ест меньше ресурсов, Ваша же решение с 2 мя переменными есть в 2 раза больше ресурсов с тем же результатом, что имхо не кошерно.
Так что в целом мои правки больше чем в 2 раза (с учётом языковых переменных во много раз) экономят ресурсы сервера с тем же визуальным и работоспособным результатом.

О каких запросах речь? Вроде итак ничего лишнего не было.

Собственно на большинство ответил выше, кроме переноса скрипта из отдельного файла в само тело overall_footer_after.html, чтобы из за незначительного текста лишний раз не мучить сервер поиском и подгрузкой дополнительного файла.

Ещё раз повторю это сугубо моё имхо не претендующие на конечную истину, и для меня важна минимальная нагрузка на сервер, при максимальном достигнутом результате.

[Perfecthus]

Фигурная скобка должна быть на новой строке

Это ваше мнение, где сказано, что должно быть с новой строки? Лично я привык писать код вот по таким стандартам.

index.html

Пустой index.html кладется для безопасности и никаких лишних запросов не создаёт.

Закрывающий тег ?> ненужен

Согласен, можно и без него обойтись, но бывают моменты, что без закрывающего тэга случаются баги, поэтому привычка закрывать осталась.

За лишние файлы сказать ничего не могу ибо просто взял файлы с другого расширения, а как писал выше еще не совсем в теме написания расширений к phpBB.

[Alecto]

Ну капец. Вам подсказали как правильно сделать и где ошиблись, а вы вместо благодарности и исправления ошибок продолжаете дурака валять и пыжиться. Лучше продолжайте обсуждать мастерство переустановки винды.