feat: глобальное переключение Fenom/MODX в code-group - #683
feat: глобальное переключение Fenom/MODX в code-group#683GulomovCreative wants to merge 1 commit into
Conversation
|
Привет @GulomovCreative! Функционал очень полезный — в документации 78 code-groups, и глобальное переключение синтаксисов сильно улучшит UX. Нашёл несколько проблем в текущей реализации:
Лучше вызывать на верхнем уровне composable.
После смены роута DOM может быть не готов. Стоит добавить nextTick. Предложение по рефакторингу Готов помочь с доработкой или могу сам оформить PR с исправлениями, если нет времени. |
Доработка #683: sync через radio+.active без click (без scroll jump), cleanup listener, useRouter на верхнем уровне, nextTick/onContentUpdated. Определение синтаксиса по language-* классу блока, а не по тексту таба.
61f2cd7 to
2a4cd5b
Compare
|
Ветка Кратко по фиксам:
Готово к ревью. @GulomovCreative |
Summary
::: code-groupчерезlocalStorage(vitepress:default-syntax)..vitepress/, неdocs/.vitepress/).Что изменено
useSyntaxSwitcherв.vitepress/theme/composables/syntax-switcher.tsDocsLayout.vueinput.checked+ класс.activeбез.click()— VitePress больше не вызываетscrollIntoViewна каждой синхронизированной группеlanguage-fenom/language-modxна блоке (работает и при кастомных заголовках табов вроде[myForm.tpl])clicklistener вonUnmounteduseRouter()на верхнем уровне composablewatch+nextTick, плюсonContentUpdated(dev/HMR)Test plan
/components/msviewcounter/frontend/catalog)pnpm dev— preference сохраняется