-
Notifications
You must be signed in to change notification settings - Fork 65
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
Вопрос про меню + баг в сафари #1298
Comments
P. S. В моём браузере сбрасывается, потому что браузерный баг. В других всё норм. |
Таня права, на главной при нажатии на ESC из активного поля курсор не убирался. Ещё можно посмотреть в превью старых ПРов, например, здесь. Хотя звучит интересно, можно было бы добавить вместе с подсказкой о клавише возможность сделать инпут неактивным.
У меня есть гипотеза, что про хоткей
По идее это подсказки о хоткеях, они не предполагаются интерактивными и не свёрстаны таковыми. На главной клик по "Клавише /" срабатывает, потому что на самом деле клик приходится по инпуту))) У меня нет принципиальной позиции по этому вопросу, если это будет востребовано, можно расширить область клика.
*звуки испорченной флейты* Вот это НАДО лечить, конечно. |
Safari и их неторопливость по имплементации CSS. Моя гипотеза, что дело в комбинации |
Про подсказки про клавиши: возможно надо как-то уменьшить аффорданс. Типа сделать чуть незаметнее как-то, чтобы не выглядело как кнопка. Но тут я фиг знает, как быть. Я бе не стала делать подсказки кликабельными (и частью кнопки). А совсем убирать подсказки не хотелось бы. Почему не хочется вкладывать подсказки в кнопку, как раньше? Дело в том, что технически клавиша / никак не связана с бургерной кнопкой. Она связана полем поиска, куда попадает фокус после этого (то есть это шорткат для быстрого перехода к поиску, а не открытия меню). С Esc тут уже ближе, да, кнопка действительно закрывает меню и этот шорткат для её срабатывания. |
Про первые три пункта это больше пообсуждать. Провёл дополнительные тесты на меню с ошибкой: Тестировал в Safari Версия 17.6 (19618.3.11.11.5) Есть расширение Касперского для паролей на своём личном компе. Ошибка есть. Тестировал в такой же Safari на рабочем, где нет расширений, ошибка есть. Тестировал в Firefox на Mac версии 126.0.1 без расширений, и на версии 129.0.2 на разных маках, ошибка есть. |
@TatianaFokina @skorobaeus закроем или в дискуссии переведём? |
@furtivite, я бы просто завела/изменила ишью и оставила лейбл #вопрос. Дискуссии у нас как-то не взлетели. |
Спасибо за огромную работу, которую вы сделали с меню) Всё стало красиво и удобно.
Подскажите, пожалуйста, возможно я что-то не понял.
Можно ли вернуть кнопку ESC и её работоспособность на главную страницу. Если я привык закрывать поиск через ESC, было бы приятно, иметь возможность убирать курсор из активного поля меню и на главной. Да, я понимаю, что на главной меню не закрывается.
На главной странице кнопка / теперь подписана словом "Клавиша /"
При этом клавиша ESC не подписана как "Клавиша ESC", а подписана просто "ESC". Можно ли написать "Клавиша ESC"?
Поймал себя на мысли, что для открытия поиска я иногда тыкаю на "Клавиша /", а не на три точки в статьях и в разделе. Можно ли сделать так, чтобы при клике на "Клавиша /" меню тоже открывалось (как на главной, на главной такое поведение)? А при клике на "ESC" соответственно закрывалась.
Словил баг при скролле в Safari, в Хроме не воспроизводится. При скроле, меню уезжает вниз.
2024-08-30.19.00.32.mov
The text was updated successfully, but these errors were encountered: