Skip to content

fix(vue): space category tree checkbox from its label - #557

Merged
biz87 merged 2 commits into
betafrom
fix/issue-555-category-checkbox-gap
Aug 16, 2026
Merged

fix(vue): space category tree checkbox from its label#557
biz87 merged 2 commits into
betafrom
fix/issue-555-category-checkbox-gap

Conversation

@Ibochkarev

@Ibochkarev Ibochkarev commented Aug 13, 2026

Copy link
Copy Markdown
Member

Описание

Во вкладке «Категории» товара подпись стояла вплотную к чекбоксу. У строки уже был gap: 0.4rem, визуально его съедал бокс PrimeVue Checkbox.

Отступ 0.5rem на label сразу после чекбокса (+ селектор), flex-shrink: 0 на чекбоксе. Строки без галочки не сдвигаются. Тот же виджет в дереве опций.

Тип изменений

  • Исправление бага (non-breaking change)
  • Новая функциональность (non-breaking change)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Рефакторинг (без изменения функциональности)
  • Документация
  • Другое (опишите):

Связанные Issues

Closes #555

Как это было протестировано?

Красный → зелёный: php tests/ResourceCategoryTreeCheckboxGapTest.php падал без margin-inline-start, после правки exit 0.

php tests/ResourceCategoryTreeCheckboxGapTest.php   # exit 0
php -l tests/ResourceCategoryTreeCheckboxGapTest.php  # exit 0
npx eslint src/components/ResourceCategoryTree.vue    # exit 0
composer test:smoke                                   # exit 0, 71 tests
  • Ручное тестирование
  • Автоматические тесты (composer test:smoke, npx eslint на изменённом Vue)
  • Тестирование на разных версиях PHP/MODX

Конфигурация тестирования:

  • MiniShop3: 1.12.0-beta1
  • MODX: 3.x
  • PHP: 8.2+

Скриншоты (если применимо)

image

Чеклист

  • Код соответствует стилю проекта
  • Добавлены/обновлены комментарии в сложных местах
  • Изменения не ломают существующую функциональность
  • Лексиконы добавлены на двух языках (ru/en)
  • PHPStan проходит без новых ошибок (composer stan / CI job PHPStan)
  • ESLint проходит без ошибок (npx eslint на ResourceCategoryTree.vue)
  • Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

Refactoring UI: отступ 0.5rem (8px) вместо произвольных 0.4rem. Ревью code-reviewer и thermo-nuclear: блокеров нет. После ревью margin только на checkbox + label.

PrimeVue Checkbox's box overflows the flex item, so row gap did not
show. Put 0.5rem on the label that follows the checkbox.
@Ibochkarev Ibochkarev added the bug Something isn't working label Aug 13, 2026
Vite emits tree styles as a shared chunk. product-tabs.min.css does
not include them, so the checkbox gap never reached the Categories tab.
@Ibochkarev
Ibochkarev requested a review from biz87 August 13, 2026 17:25
@biz87
biz87 merged commit a0d0d51 into beta Aug 16, 2026
3 checks passed
@biz87
biz87 deleted the fix/issue-555-category-checkbox-gap branch August 16, 2026 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Нет отступа между чекбоксом и названием категории

2 participants