Skip to content

Instantly share code, notes, and snippets.

@yoksel
Last active March 13, 2022 18:26
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save yoksel/b05fc45764dd1e35afa22336ae26cf34 to your computer and use it in GitHub Desktop.
Save yoksel/b05fc45764dd1e35afa22336ae26cf34 to your computer and use it in GitHub Desktop.

Бухвалова Юлия. Комментарии к проекту.

Ссылка на проект: https://yoksel.github.io/A11y-course/

Свёрстаны все элементы, добавлены состояния. Системный фокус заменён на ярко-зелёный, потому что синий системный сливается с фиолетовыми кнопками.

Мобильной версии и версии без JS нет, не успела сделать.

Изменения в дизайне при вёрстке

  1. В фиолетовых кнопках поменяла чёрный текст на белый, чтобы его было видно.

  2. Убрала подчёркивание к кнопок табов. Предположила, что для содержимого табов нет отдельных страниц, следовательно, кнопки табов не являются ссылками, поэтому подчёркивания там быть не должно. Возможно, это имело бы смысл для версии без JS, но у меня такой версии нет.

  3. Задала подчёркивание всем ссылкам на странице, чтобы ссылки в тексте отличались не только цветом, и там, где ссылки покрашены в цвет текста, было бы видно, что это ссылки.

  4. Увеличила расстояние между поиском и переключением языков, чтобы было понятнее, что объекты не связаны.

  5. Увеличила логотип, иначе текст на нём читается с трудом.

  6. Для случаев, когда кнопки переключают состояния (табы, языки), сделала состояние активной кнопки и неактивной, чтобы было понятно что сейчас выбрано.

  7. Навигацию над слайдером унесла в шапку, сделала ссылками на разделы страницы и соответственно поменяла порядок. По-моему, это навигация по странице, и она должна быть в шапке, а не рядом со слайдером.

  8. Добавила кнопку закрытия в модальное окно.

  9. Для заполненных полей и полей в фокусе лейбл подпрыгивает вверх, чтобы он всегда был виден. Исключение — поле поиска. В нём нет плейсхолдера и я предположила, что иконка помогает понять что это за поле. Не уверена что это верное решение.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment