Skip to content
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

Open
furtivite opened this issue Aug 30, 2024 · 7 comments
Open

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

furtivite opened this issue Aug 30, 2024 · 7 comments
Labels
вопрос Требуется обсуждение вёрстка Задачи по вёрстке ошибка Что-то критичное и мешает улучшение Доработка существующего

Comments

@furtivite
Copy link
Member

furtivite commented Aug 30, 2024

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

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

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

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

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

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

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

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

2024-08-30.19.00.32.mov
@furtivite furtivite changed the title Вопрос про меню Вопрос про меню + баг в сафари Aug 30, 2024
@furtivite furtivite added ошибка Что-то критичное и мешает улучшение Доработка существующего вопрос Требуется обсуждение вёрстка Задачи по вёрстке labels Aug 30, 2024
@TatianaFokina
Copy link
Member

TatianaFokina commented Aug 30, 2024

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

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

@skorobaeus
Copy link
Member

skorobaeus commented Aug 30, 2024

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

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

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

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

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

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

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

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

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

@TatianaFokina
Copy link
Member

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

@TatianaFokina
Copy link
Member

TatianaFokina commented Aug 30, 2024

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

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

@furtivite
Copy link
Member Author

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

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

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

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

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

@furtivite
Copy link
Member Author

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

@TatianaFokina
Copy link
Member

@furtivite, я бы просто завела/изменила ишью и оставила лейбл #вопрос. Дискуссии у нас как-то не взлетели.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
вопрос Требуется обсуждение вёрстка Задачи по вёрстке ошибка Что-то критичное и мешает улучшение Доработка существующего
Projects
None yet
Development

No branches or pull requests

3 participants