Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Нужны хуки #22

Closed
vimruler opened this issue Feb 12, 2015 · 7 comments
Closed

Нужны хуки #22

vimruler opened this issue Feb 12, 2015 · 7 comments

Comments

@vimruler
Copy link

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

  1. В файле \application\maxsite\admin\plugins\admin_page\form.php есть хук admin_page_form_add_block_1, который позволяет добавить свой код на вкладке «Прочее». Но это дабовление осуществляется перед всеми остальными полями вкладки. Хотелось бы иметь возможность добавлять свой код и после всех стандартных полей. Т.е. нужен хук типа admin_page_form_add_block_2, который расположен после поля «Дата публикации». Это должно быть примеро в строке 258 сейчас.

    Для примера покажу как у меня делается сейчас:
    Скриншот

  2. Иногда есть необходимость добавить кнопки комментатору. Обычно уже работает плагин comment_button. Но часто (из-за шаблона) приходится извращаться со стилями, чтобы рядом с кнопками comment_button поставить свои кнопки. Да и писать целый плагин ради нескольких похожих кнопок как-то избыточно. Хотелось бы иметь хук в плагине comment_button, чтобы можно было через ушки-то-хук добавлять в ряд кнопок свою кнопку (или несколько).
    Если с этим пунктом поддержите, то могу реализовать не только хук, но и настройки плагину, чтобы была возможность включать-выключать нужные кнопки.

@maxsite
Copy link
Owner

maxsite commented Feb 15, 2015

По хуку после даты. Там проблема в том, что в последнем элементе используется p.ends который убирает горизонтальную линию-разделитель. Если добавлять новый пункт, то придется как-то учитывать эту особенность. Сейчас я понимаю, что нужно было делать через HR, тогда все элементы были бы едины. Но сейчас менять — потянется хвост в стили админки. Мы можем просто пропиать хук, если это действительно нужно после даты, не задумываясь над его оформлением. Если устраивает, я сделаю.

По comment_button я толком не понял, что там нужно. Илья, вы можете просто сделать новую версию этого плагина.

Оффтоп. Кто знает где на гитхабе автоматом подписываться на новые issue?

@cuprum
Copy link

cuprum commented Feb 15, 2015

@maxsite А тебе разве не приходят?
Проверь настройки
уведомления

@maxsite
Copy link
Owner

maxsite commented Feb 15, 2015

Да, так и отмечено. Но это для тех, где я участвую. А если кто-то делает новый тикет, сообщений по email нет.

@cuprum
Copy link

cuprum commented Feb 15, 2015

@maxsite А вот тут так?

watch

@vimruler
Copy link
Author

Там проблема в том, что в последнем элементе используется p.ends который убирает горизонтальную линию-разделитель. Если добавлять новый пункт, то придется как-то учитывать эту особенность. Сейчас я понимаю, что нужно было делать через HR, тогда все элементы были бы едины. Но сейчас менять — потянется хвост в стили админки. Мы можем просто пропиать хук, если это действительно нужно после даты, не задумываясь над его оформлением. Если устраивает, я сделаю.

Проблема понятна. Думаю, что можно всё таки добавить хук, но в коде оставить камент с описанием ситуации для разработчиков.

По comment_button я толком не понял, что там нужно. Илья, вы можете просто сделать новую версию этого плагина.

Хорошо. Пришлю свой код, когда реализую.

@maxsite
Copy link
Owner

maxsite commented Feb 16, 2015

@cuprum Ага, тут не стояла отметка. Поставил.

@vimruler Ок. Хук добавлю.

@maxsite
Copy link
Owner

maxsite commented Feb 20, 2015

Обновил и плагин.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants