-
Notifications
You must be signed in to change notification settings - Fork 0
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
Мысли, пожелания по модулю #1
Comments
Раскрой мысль, plz. Что значит "переопределения шаблона с использованием другого шаблонизатора"? |
Согласен. Но для тех кто не переопределяет View, такой функционал был бы полезен. Правда пока еще не думал над реализацией. |
Вероятно, таким людям следует просто переопределить метод Pagination::render() под свои нужды. Для бОльшего удобства можно вынести все, что завязано на рендеринг шаблона, в отдельный метод _render(). То есть в render() останутся проверки, общие для любого шаблонизатора, а в _render() можно будет переопределить чисто свои хотелки. |
Ага согласен. Чего тогда не хватает в том модуле ? |
Посмотрел пагинацию в Yii. В шаблонах используют вывод через виджет. |
Это простой, несложный модуль. Не надо к нему прикручивать никаких лишних наворотов. ИМХО, на данный момент вполне достаточно довести до ума kohana-pagination (версии для 3.2 и 3.3), актуализировать стандартные шаблоны для пагинатора (взять за основу самые популярные + какие-нить привычные из соцсетей + подсмотреть в других фреймах). Сперва нужна основа - просто работающий модуль. Далее будем собирать фидбеки, нужно будет что-то допилить - пользователи сами скажут об этом. |
Приветствую. Касаемо Pagination обычно все что включает релизация этого модуля - вывод самой пагинации. Ни у кого не было идеи чтобы этот модуль так же выполнял работу с данными? Т.е. на входе получал array($orm, $per_page, $current_page) и возвращал array($results, $total_pages) |
Еще один момент, на текущий момент чтобы kohana-pagination корректно работал, необходимо чтобы для был отдельный роут для текущего контроллера. В админке с общем роутом под неё, kohana-pagination берет только первый сегмент url - /admin/. |
Зачем завязываться на определенный модуль (который еще и не является обязательным)? Тогда уж пусть модель ORM подготавливает данные для пагинатора.
Можно поподробнее? А лучше - в отдельный тикет. |
Да, на счет ORM согласен. Еще предложение ввести число максимального кол-во дополнений, для того чтобы на последнюю страницу не приходилось 1-2 результата. Подробнее тут: http://habrahabr.ru/qa/3198/ |
Идею понял. Тикет, please )) |
|
Можно сделать абстрактную модель ORM, от нее наследовать свои Model_Article, Model_News и тд |
Есть модуль https://github.com/kloopko/kohana-pagination
Не хватает переопределения шаблона с использованием другого шаблонизатора.
Что еще ?
The text was updated successfully, but these errors were encountered: