pdoTools icon indicating copy to clipboard operation
pdoTools copied to clipboard

Предлагаю изменить поведение ajaxElemWrapper в pdoPage.

Open Alexij2 opened this issue 4 years ago • 1 comments

Добрый день. Коротко: Если параметр в pdoPage ajaxElemWrapper пуст, то пусть берется родитель от ajaxElemRows исправление в 123 строке js файла на: var wrapper = config['wrapper'] ? $(config['wrapper']) : $(config['rows']).parent(); https://github.com/sergant210/pdoTools/blob/e21ee5373f945e564b9971a40b3f2118596fe85b/assets/components/pdotools/js/pdopage.js#L123 Для чего? Чтобы не прописывать его явно и делалось все автоматом (универсально). Т.к. иногда возникают сложности прописать и придумать лишний класс, для того чтобы просто скролилась страница вверх при нажатии пагинации. PR не делал т.к. выношу на просьбу и обсуждение.

Alexij2 avatar Sep 05 '21 19:09 Alexij2

Для чего? Чтобы не прописывать его явно и делалось все автоматом (универсально). Т.к. иногда возникают сложности прописать и придумать лишний класс ...

А нельзя просто использовать всё по-умолчанию? Чтобы ничего не придумывать.

П.С. Я этим не пользуюсь, поэтому не обладаю экспертным мнением.

sergant210 avatar Sep 06 '21 07:09 sergant210