fix(mgr): визуальная согласованность Vue-менеджера с MODX - #621
fix(mgr): визуальная согласованность Vue-менеджера с MODX#621Ibochkarev wants to merge 3 commits into
Conversation
|
@biz87 Не рассматриваем пока - посмотрю, что останется после новой темы vueTools |
… MODX Conflict resolution: keep array-based fieldsBySections (merged modx-pro#620/modx-pro#611), port fieldHtmlId labels + isCheckboxXtype/normalizeCheckboxValue helpers on top.
Conflict resolution: PR 640 supersedes merged modx-pro#621 rework (same author, same intent) — took PR side for 36 files; manually preserved modx-pro#631 useConfirm grids, modx-pro#623 datefield dialog styles, modx-pro#643 gallery bits, modx-pro#605 order entry; ProductData sections rebuilt on groupProductDataSections (keeps modx-pro#611/modx-pro#620 sort_order) under PR 640 Panel layout.
|
Этот PR включён в тестовую интеграционную сборку всех открытых PR MiniShop3: AgelxNash/MiniShop3, ветка Сборка нужна, чтобы проверить совместимость взаимозависимых серий PR до их мержа — при последовательном слиянии они конфликтуют друг с другом. Это не ревью и не конкурирующий PR: авторство сохранено (1 PR = 1 коммит с исходным автором), ветка пересобирается по мере обновления PR. Как вошёл в сборку: Слит в ветку раньше #640. Файлы, перекрываемые более новым реворком #640, взяты из #640; наработки #621 (fieldHtmlId-лейблы, |
|
Привет! Просто пожелание: удачи с этим PR 🚀 Работа нужная — пусть рассмотрят и смержат как можно скорее. Успехов! |
Unify primary/success to AA-safe #4e8136 (preset + CSS var inject for VueTools-external PrimeVue), flatten ProductDataFields Card/for-id, fix option_fields under ONLY_FULL_GROUP_BY, and polish Utilities/Uppy/ vtabs/a11y/lexicons (#615).
Match MODX primary green, unify toolbar/tab spacing, and make gallery upload and bulk actions clearer and keyboard-friendly.
Force buttons and AutoComplete dropdowns to the shared 2.25rem control height, and fix product links search/dialog spacing inside .vueApp.
7bf2980 to
5d733a6
Compare
Описание
Vue Manager выглядел «инородным» в MODX: несколько зелёных + синий accent Aura, Card «Данные товара», сломанные
for/id, пустая вкладка «Опции» приONLY_FULL_GROUP_BY, прыгающие отступы, разная высота кнопок/полей и слабая affordance галереи.Этот PR закрывает пакеты A–D из #615 и follow-up polish после ревью live UI.
Primary (MODX chrome)
emerald/greenremapped на шкалу с якорем#6cb24a(как ExtJS.primary-button), hover#528738.injectMs3ThemeVars) — нужен, потому чтоprimevueexternal из VueTools и runtime Theme оставлял#10b981.#4e8136: приоритет визуального совпадения с менеджером MODX (white-on-primary как в core).Плотность и контролы
2.25rem(36px) для Input/Select/Button/size="small"/lg, AutoComplete dropdown — черезinjectFormStylesOverride(MS3_CONTROL_HEIGHT).--ms3-tab-panel-padding, без двойных padding у children.Галерея
Связи товара
InputText(не raw<input>).append-to="self"чтобы scoped.vueAppстили работали на teleport.Прочее из #615
for/id(fieldHtmlId).AdminOptionFieldsGROUP BY подONLY_FULL_GROUP_BY+ smoke-тест.Тип изменений
Связанные Issues
Closes #615
Refs #616 (compact Uppy / gallery UX; сортировка upload по имени — вне scope)
Коммиты
177652d4— align Vue Manager visuals with MODX chrome (theme, ProductData, options GROUP BY, utilities/a11y)10b51622— polish density and gallery affordances (primary#6cb24a, gallery toolbar/upload/a11y)7bf29808— unify control heights and product links form layout (formStyles+ Links dialog)Как это было протестировано?
Live
project.testподroot:for==df-field-*; primary#6cb24a#6cb24aКонфигурация тестирования:
fix/issue-615-vue-modx-visual-consistencyЧеклист
Дополнительные заметки
Deferred (non-block): полный overlay category attrs через CaptionOverlayResolver вместо SQL aggregates; Checkbox
:valueвсё ещё object (accessible name исправлен). Dialog’ы с teleport вbodyпо-прежнему требуютappend-to="self"(или глобальных стилей без.vueApp), иначе scoped padding/gap не применяются.