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

Правит ссылку #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

furtivite
Copy link

https://weblind.ru/inner.html#nav-menu

Если кликнуть на любой пример в меню «Шапки», «Цены», «Контакты», то сваливаешься не на https://weblind.ru/inner.html даже, а на https://weblind.ru и это несколько озадачивает.

Стандарт, кажется, тоже разрешает ссылки без href=""

@maniyax
Copy link
Contributor

maniyax commented May 31, 2021

Приветствую!

Если вы создаете ссылку без href="", то она перестает быть фокусируемой скринридером.

Убирать href="" из пунктов меню ни в коем случае нельзя в показанном примере. Даже у текущей страницы.

@furtivite
Copy link
Author

furtivite commented May 31, 2021

Приветствую!

Если вы создаете ссылку без href="", то она перестает быть фокусируемой скринридером.

Убирать href="" из пунктов меню ни в коем случае нельзя в показанном примере. Даже у текущей страницы.

Спасибо за фидбэк! Нормально было бы, если ссылка была бы <a href="/inner.html#nav-menu">? Визуально страница немного «прыгнет», но останется на том же месте.

@maniyax
Copy link
Contributor

maniyax commented May 31, 2021

Вообще странно, что href="#" адресует на главную. Ссылка должна была просто стать посещенной.

Тогда можно сделать у всех ссылок href="#nav-menu" и задать

.

@maniyax
Copy link
Contributor

maniyax commented May 31, 2021

Обернуть в <nav id="nav-menu">, и тогда будет работать с href="#nav-menu", все верно.

Спасибо, что заметили :-)

@maniyax
Copy link
Contributor

maniyax commented May 31, 2021

Осталось дождаться Вадима, чтобы он влил.

@furtivite, а можете глянуть еще #74?

Я добавлял все вслепую в Notepad++, из-за чего могло поехать форматирование по строкам. Здесь какое-то странное используется, по 100 символов, кажется.

А то у Вадима, похоже, пока руки не доходят поправить и влить.

@furtivite
Copy link
Author

@maniyax вроде сейчас всё сделал так, как надо, ваш пиар пойду смотреть тоже)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants