-
Notifications
You must be signed in to change notification settings - Fork 80
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
Feature request: Поправить сортировку опечаток #205
Comments
Если необходимо, то готов реализовать |
Мне кажется, тут должно как-то сложнее быть, точнее без рукопашки. Я пока не знаю как реализовано, я бы предложил посмотреть, есть ли библиотечки для удобной сортировки данных. Плюс сортировка по дефолту должна быть такая - сверху старые записи, внизу свежие |
@fey Ну стрелочка как минимум точно не должна быть показана до тех пор, пока по конкретному критерию не происходит сортировка, а сейчас она показывается вообще у всех пунктов, что путает Ну и это никак не конфликтует с сортировкой по умолчанию, наоборот, помогает её визуально сразу увидеть |
@bazilval некоторые задачи могут иметь разные решения. например в Хекслете когда-то в топе пользователей было больше 100 юзеров, была пагинация, которая сильно медленно работали. Вместо оптимизации кода тупо выключили пагинацию и оставили топ 100 юзеров. В целом можно посмотреть штуку с сортировкой, но тут есть некоторые нюансы
|
тут штука в том, что htmx не изучается в профессиях. По идее можно, но, если есть реакт, то зачем?) |
@fey чтобы у джавистов было больше разнообразной практики, я думал этот проект тут для этого. У фронтендеров такой возможности и так полно, кажется. Кстати, Thymeleaf теперь тоже нет в профессии :) Пулл-реквест со стрелками сделаю, но после мержа предыдущего, потому что там те же строчки редактируются |
@fey да, по идее надо проект обновлять под новые технологии. Например и мавена нет - надо на градл переходить. |
@fey ну как попытка оставаться в той же экосистеме, которая уже написана, просто добавить интерактивности с помощью HTMX. Особенно до того момента, пока нет ничего другого. Ведь добавление фронта на React - это как раз создание отдельной кодовой базы, полная переработка существующего представления и так далее. Как-будто более многодельное что-то получается Но опять же, если стоит задача добавить практики фронтам, то могу понять :) |
Ну тут такая штука, что java это больше работа с апи, чем с шаблонизатором. Просто в начале было проще сделать на шаблонизаторе все, чтобы было с чем работать. |
Понимаю, да. Но по сути это просто другой подход к фронтенду, скорее развитие шаблонизации (которую до прихода реакта, кстати, тоже в любом случае придётся поддерживать). Подумал, что было бы здорово попробовать что-то такое. Некоторые наоборот с реакта на это добро переходят :) Возможно сделаю на своём форке, чтобы было на что посмотреть и пообсуждаем |
да, в некоторых проектах нет реакта, точнее он есть, но точечно. Например хекслет на 99% - это классический бекенд. |
Problem description
Стрелочки у заголовков столбца отображаются постоянно, хотя должны быть только тогда, когда по конкретному столбцу необходима сортировка
![image](https://private-user-images.githubusercontent.com/29205112/270011754-06759ce0-431e-4770-9081-6bdf35e49b63.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQyOTExMzIsIm5iZiI6MTcyNDI5MDgzMiwicGF0aCI6Ii8yOTIwNTExMi8yNzAwMTE3NTQtMDY3NTljZTAtNDMxZS00NzcwLTkwODEtNmJkZjM1ZTQ5YjYzLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODIyVDAxNDAzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTk4MWM2MWRkYWIyZTZjYjVhNDU4ZmE5MDIyZjZiMWY0ZDUxODgzYjg1ZDJiMmUxMjU2ZTBlMjI0MWI4MGQ3YmMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.d1Jh7_Sf-Naa07l2oV0JEn4F8KIfO9--gLWOTMW_PhA)
Вынести разметку заголовков с сортировкой в отдельный фрагмент, чтобы можно было прописать только что-то такое:
<th th:replace="fragments/sorting :: sorting('id','Id')"></th>
Это приведёт к снижению однотипной разметки, а также позволит переиспользовать её во всех таблицах (пользователей, воркспейсов)
Proposed solution
th:if="${sortProp == 'columnName'}"
, а внутри оставить только проверку на направление сортировки.В итоге как-то так:
<span th:if="${sortProp == 'modifiedBy'}">[[${sortDir.isAscending() ? '⇑' : '⇓'}]]</span>
Как должно выглядеть:
![image](https://private-user-images.githubusercontent.com/29205112/270011368-7796b239-2fb3-4ccc-9cf4-0cd844391c22.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQyOTExMzIsIm5iZiI6MTcyNDI5MDgzMiwicGF0aCI6Ii8yOTIwNTExMi8yNzAwMTEzNjgtNzc5NmIyMzktMmZiMy00Y2NjLTljZjQtMGNkODQ0MzkxYzIyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODIyVDAxNDAzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTE5NGY0NGMzNWI1MmIwZjQzY2U3MGU3OTk4NTU4NmMyYTE0M2JhMTU1MTZmMjAyMjRmNDRjM2QwOWNmMjhlOWYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Be7ZCHt1OR-c90fXsQCcBwTHBNesv_-im7a8rFzluBE)
Вставка фрагмента
Сам фрагмент
The text was updated successfully, but these errors were encountered: