doka-guide / platform

Платформа Доки: шаблоны, стили, скрипты и всё для сборки сайта
https://doka.guide
MIT License
233 stars 65 forks source link

Вопрос про меню + баг в сафари #1298

Open furtivite opened 2 weeks ago

furtivite commented 2 weeks ago

Спасибо за огромную работу, которую вы сделали с меню) Всё стало красиво и удобно.

Подскажите, пожалуйста, возможно я что-то не понял.

  1. Можно ли вернуть кнопку ESC и её работоспособность на главную страницу. Если я привык закрывать поиск через ESC, было бы приятно, иметь возможность убирать курсор из активного поля меню и на главной. Да, я понимаю, что на главной меню не закрывается.

  2. На главной странице кнопка / теперь подписана словом "Клавиша /"

Снимок экрана 2024-08-30 в 18 58 22

При этом клавиша ESC не подписана как "Клавиша ESC", а подписана просто "ESC". Можно ли написать "Клавиша ESC"?

Снимок экрана 2024-08-30 в 18 59 02
  1. Поймал себя на мысли, что для открытия поиска я иногда тыкаю на "Клавиша /", а не на три точки в статьях и в разделе. Можно ли сделать так, чтобы при клике на "Клавиша /" меню тоже открывалось (как на главной, на главной такое поведение)? А при клике на "ESC" соответственно закрывалась.

  2. Словил баг при скролле в Safari, в Хроме не воспроизводится. При скроле, меню уезжает вниз.

https://github.com/user-attachments/assets/25bc7a80-ff7d-4017-ba1b-3c8169b0ff31

TatianaFokina commented 2 weeks ago
  1. Егор, никто не трогал кнопку Esc. Она сейчас работает стандартно, как и везде. Когда ты на поле и жмёшь её, сбрасывается фокус. Там вообще же на главной никак не отслеживается это событие.

P. S. В моём браузере сбрасывается, потому что браузерный баг. В других всё норм.

skorobaeus commented 2 weeks ago
  1. Можно ли вернуть кнопку ESC и её работоспособность на главную страницу. Если я привык закрывать поиск через ESC, было бы приятно, иметь возможность убирать курсор из активного поля меню и на главной. Да, я понимаю, что на главной меню не закрывается.

Таня права, на главной при нажатии на ESC из активного поля курсор не убирался. Ещё можно посмотреть в превью старых ПРов, например, здесь. Хотя звучит интересно, можно было бы добавить вместе с подсказкой о клавише возможность сделать инпут неактивным.

  1. На главной странице кнопка / теперь подписана словом "Клавиша /" При этом клавиша ESC не подписана как "Клавиша ESC", а подписана просто "ESC". Можно ли написать "Клавиша ESC"?

У меня есть гипотеза, что про хоткей [/] почти никто не знал, и эта подсказка о нём была невидимой. Какая-то косая черта в квадратике, что это, зачем она вообще нужна? Слово "Клавиша" отвечает именно на этот вопрос — это кнопка! на клаве! её можно нажать! ESC и Enter кмк сразу считываются как кнопки на клавиатуре и в дополнительных пояснениях не нуждаются. Да и засорять интерфейс словом "клавиша" мне не хочется.

  1. Поймал себя на мысли, что для открытия поиска я иногда тыкаю на "Клавиша /", а не на три точки в статьях и в разделе. Можно ли сделать так, чтобы при клике на "Клавиша /" меню тоже открывалось (как на главной, на главной такое поведение)? А при клике на "ESC" соответственно закрывалась.

По идее это подсказки о хоткеях, они не предполагаются интерактивными и не свёрстаны таковыми. На главной клик по "Клавише /" срабатывает, потому что на самом деле клик приходится по инпуту)))

У меня нет принципиальной позиции по этому вопросу, если это будет востребовано, можно расширить область клика.

  1. Словил баг при скролле в Safari, в Хроме не воспроизводится. При скроле, меню уезжает вниз.

*звуки испорченной флейты* Вот это НАДО лечить, конечно.

TatianaFokina commented 2 weeks ago

Safari и их неторопливость по имплементации CSS. Моя гипотеза, что дело в комбинации sticky с чем-то ещё.

TatianaFokina commented 2 weeks ago

Про подсказки про клавиши: возможно надо как-то уменьшить аффорданс. Типа сделать чуть незаметнее как-то, чтобы не выглядело как кнопка. Но тут я фиг знает, как быть. Я бе не стала делать подсказки кликабельными (и частью кнопки). А совсем убирать подсказки не хотелось бы.

Почему не хочется вкладывать подсказки в кнопку, как раньше? Дело в том, что технически клавиша / никак не связана с бургерной кнопкой. Она связана полем поиска, куда попадает фокус после этого (то есть это шорткат для быстрого перехода к поиску, а не открытия меню). С Esc тут уже ближе, да, кнопка действительно закрывает меню и этот шорткат для её срабатывания.

furtivite commented 2 weeks ago

Про первые три пункта это больше пообсуждать. Провёл дополнительные тесты на меню с ошибкой:

Тестировал в Safari Версия 17.6 (19618.3.11.11.5)

Есть расширение Касперского для паролей на своём личном компе. Ошибка есть.

Тестировал в такой же Safari на рабочем, где нет расширений, ошибка есть.

Тестировал в Firefox на Mac версии 126.0.1 без расширений, и на версии 129.0.2 на разных маках, ошибка есть.

furtivite commented 1 week ago

@TatianaFokina @skorobaeus закроем или в дискуссии переведём?