-
Notifications
You must be signed in to change notification settings - Fork 62
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
Рефакторинг меню #1191
Open
TatianaFokina
wants to merge
111
commits into
doka-guide:main
Choose a base branch
from
TatianaFokina:menu-refactoring
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Рефакторинг меню #1191
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TatianaFokina
added
улучшение
Доработка существующего
вёрстка
Задачи по вёрстке
дизайн
Задачи по дизайну
labels
Aug 17, 2023
TatianaFokina
requested review from
furtivite,
solarrust,
igsekor and
HellSquirrel
as code owners
August 17, 2023 12:03
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as duplicate.
This comment was marked as duplicate.
1. На мобильных этот попап закрывает дропдаун, это нехорошо 2. Вызывая какую-то функцию в интерфейсе, пользователь отдаёт ей максимальный приоритет в своём внимании и ждёт, что она будет прямо перед ним. Странно прятать только что вызванное меню под попапом
Оставлю черновик чеклиста для сценариев тестирования пока тут. Управление с клавиатурыВсе страницы:
Все страницы, кроме главной и расширенного поиска:
Главная:
Расширенный поиск:
Управление мышкой и тапамиВсе страницы, кроме главной и расширенного поиска:
Страницы для тестирования |
Потому что они сводили меня с ума
Заменяет иконку с крестиком на иконку с бэкспейсом на странице поиска
Превью контента из e166121 опубликовано. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Превью контента.
Пока-пока, #1263, #1110, #1171, #819, #1074, #1069, #1075, #1081.
Ждут своего героя/героиню 🦸
Скорее всего для @skorobaeus:
Сноски ¹²³
¹ У поля поиска одинаковый невидимый лейбл, плейсхолдер и подсказка к горячей клавише с названием «Поиск». Это не какой-то финальный вариант, просто хотела проверить гипотезу про то, что слишком много поисков, и это может запутать.
² Дефолтная выглядит странно, но может и нет. Изменила у .search
font-size
иline-height
(818d626, eb44a0b).³ Дело в том, что на этой странице у меню нет дропдауна, а строка поиска видна всегда. Кнопка с крестиком, которая у нас нужна для закрытия меню, здесь не закрывает меню, а очищает фильтры + поле поиска, если там есть запрос. Описано в #1074.
Скорее всего для @igsekor:
Скорее всего для @solarrust или @furtivite:
См. padding
Для @TatianaFokina:
aria-controls
иaria-expanded
к кнопке.Ченджлог 🥵
Стили в общем
--color-fade
с видаhsl(220 7% 25%)
стал220 7% 25%
, чтобы добавлять альфа-канал при затемнении ⁴--color-border: hsl(var(--color-base-text) / 0.3)
, чтобы убрать повторы из кода.⁴ Иначе фильтры не работают.
Структура хедера
Другой текст у плейсхолдера в поле поиска, чтобы не повторялось два раза слово «Поиск».⁵ Подсказка про поиск раньше была вложена внутрь
<button>
, что мне показалось странным. Гамбургер отвечает не только за поиск, но и за выпадающее меню с разделами. Плюс на слух довольно странно будет сначала услышать что-то про поиск, потом про то, что эта кнопка открывает и закрывает меню. Так что самое логичное место для шортката, конечно, поле поиска, но тут у нас оно скрыто, так что ситуация особая.ARIA
aria-expanded
(скринридер сообщает, свёрнут или развёрнут сейчас дропдаун) иaria-controls
(связывает кнопку с дропдауном).aria-shortcuts
с кодом /.aria-label
для основного хедера (ориентир).aria-expanded
иaria-controls
у кнопки для скрытия и показа оглавления к материалам на мобилках.Поведение хедера
Slash
, но иnumpadDivide
(потестировать отличия в JavaScript Key Code Event Tool).Стили хедера
<body>
).user-select: none
.isSticky
всегда56px
⁶⁶ Удалила функцию для расчёта высоты. Она у нас не такая вариативная, поэтому кажется, что 56 пикселей во всех случаях вполне норм. Так мы избавимся от лишних вычислений и не будем рассчитывать каждую секунду текущую высоту хедера.
Содержание материалов
Хедер отдельного материала
Новое
Скриншоты с изменениями
Оффсеты у стикящихся элементов.
Случился упс 🫠
Решённое
На странице "Участники" и странице каждого участника меню сжимается позже, чем этого ожидаешь. Интересно, получится ли стриггернуть сжатие сразу, как начинается текст? UPD: Решено сжатием хэдера сразу при скролле
bandicam.2024-05-22.21-46-22-881.mp4
Не все элементы со страниц затемняются при развёрнутом хедере.
Поиск на главной: нужен ли там подсказка на мобилках вообще? UPD: Нет, не нужна.
Рамка остаётся при раскрытии дропдауна, убрать. UPD: Таня — моя героиня!
Когда фон хедера и дропдауна как у страницы, они сливаются с другими элементами.
Сейчас попап появляется поверх открытого дропдауна.
Из-за того, что появился текст про поиск, на мобилках на новую строку переносятся хлебные крошки.
UPD: Нужно убрать хлебные крошки на индексах разделов, на проде их нет, и они не нужны, т. к. дублируют инфу.
UPD2: Убрала
Из-за анимации (а может и нет) и того, в какой момент убирается класс для сужения высоты и отступов в меню, оно в определённой точке скролла начинает паниковать и трястись. Воспроизводится только на странице материала.
_blockquote_.HTML.-.Google.Chrome.2024-05-09.15-19-05.mp4
Похожий упс на странице участника:
bandicam.2024-05-22.22-03-53-328.mp4
Нужна другая иконочка, не с крестиком, как у нас сейчас.
UPD: Сменила иконку на "бэкспейс"
Фильтры на странице всех участников на мобилках.
UPD: Сейчас кнопка фильтров скрывается под меню, эти сущности исключают друг друга
Затемнение под фильтрами на странице участников и затемнение под меню это одна и та же механика, должны выглядеть одинаково:
Бэклог 😶🌫️
Медиафичу для отключения анимаций выносим в отдельный PR, а то и так многаааа. Кнопку «Наверх» тоже выносим.