phpBB Guru - Официальная русская поддержка форума phpBB3

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

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

Позже хочу добавить админку. Писал плагин прокрутки для движка е107, но там совершенно другое API, с phphBB пока так сильно не знаком, поэтому админки пока нет.
В планах добавить в админку следующие настройки:
Установка: Распаковать в каталог: root/ext/pwg/pagescrolling/ и активировать расширение в админпанели Персонализация -> Управление расширениями

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

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

staiki писал(а): Можно коричневый или чёрным?

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Perfecthus писал(а): Вроде итак ничего лишнего не было

Лишние файлы ext.php и index.html
listener.php Этот кусок кода линий. Фигурная скобка должна быть на новой строке Аналогично. Три пустых табулятора в конце строки Фигурная скобка должна быть на новой строке
После закрывающей фигурной скобки лишний табулятор
PageScrolling.php
Строки 21 и 25 - Фигурная скобка должна быть на новой строке
Строка 32 - пустая
Закрывающий тег ?> ненужен

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

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

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

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

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

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

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

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

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

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

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

Sheer писал(а): index.html

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

Sheer писал(а): Закрывающий тег ?> ненужен

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

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

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