Skip to content

Add sort options and rating column to Installer package browser - #90

Open
Ibochkarev wants to merge 2 commits into
3.xfrom
fix/14845-installer-addon-sorting
Open

Add sort options and rating column to Installer package browser#90
Ibochkarev wants to merge 2 commits into
3.xfrom
fix/14845-installer-addon-sorting

Conversation

@Ibochkarev

@Ibochkarev Ibochkarev commented Aug 21, 2026

Copy link
Copy Markdown
Member

Что сделано

  • Дефолтная сортировка Most Downloads в грид и thumbs-видах
  • Тулбар использует существующий лейбл sort_by_dots; сортировка перезагружается только на select
  • Рейтинг приходит из списка пакетов провайдера в грид, карточки thumbs и панели деталей
  • Дублирующие id колонок (author-col, downloads-col) исправлены

Зачем нужно

modxcms#14845 просил видимый контроль сортировки и колонку рейтинга в браузере пакетов Installer.

Как проверить

  1. Откройте Extras → Install → Add Extras
  2. Подтвердите лейбл сортировки и дефолт Most Downloads
  3. Смените сортировку в грид и thumbs: список перезагружается один раз
  4. Подтвердите Rating в колонке грида, meta thumbs и деталях пакета
  5. Подтвердите, что пустой рейтинг показывает --

Связанные

Refs modxcms#14845

Оригинал: modxcms#16828

Ibochkarev and others added 2 commits July 27, 2026 18:46
- Add 'Sort by' label and default sort (downloads) for grid and thumbs view
- Add rating column to package grid
- Add rating field to modTransportProvider::find()
- Add lexicon entries: rating, sort_by
- Fix duplicate column IDs (author-col, downloads-col)
- Add change handler for sort combobox in thumbs view
- Use single change event to prevent double load

Co-authored-by: Cursor <cursoragent@cursor.com>
Show rating in thumbs and details, listen only to select for sort, and reuse sort_by_dots instead of a new lexicon key.
@Ibochkarev Ibochkarev changed the title Опции сортировки и колонка рейтинга в браузере пакетов Installer Add sort options and rating column to Installer package browser Aug 21, 2026
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

Successfully merging this pull request may close these issues.

1 participant