helena-herasymiuk / uber_eats Goto Github PK

Загрузка шрифта одна из самых больших проблем сайта. Используйте выборку диапазонов символов и не забывайте про свойство font-display

Стандартный viewport задается так:
<meta name="viewport" content="width=device-width, initial-scale=1.0">

Пожалуйста, не отключайте zoom для пользователя. Пользователь должен иметь возможность изменять zoom, как минимум до 200%. Это прописано в стандарте по доступности.

Подробнее:

• Don't Disable Zoom
• How to disable viewport scaling in iOS 10? You don't.
• Proposal: devs SHOULD NOT use content="user-scalable=no | maximum-scale=1.0 " on meta · Issue #602 · w3c/html

Исходя из каких соображений вы здесь выбрали span?

Есть базовые правила вложенности элементов, которые у тебя явно нарушены уже в структуре. Что очень сильно режет глаз, даже если проблема будет исправлена на уровне CSS.

Заголовки это очень важно

А тебя тут только четвертый уровень присутствует... И почему 4-й, если первых трех нету...

Размер карточки у тебя зависит от размера фотки. А такого не должно быть в принципе. Ибо:

1. У тебя контент начинает "прыгать" на медленном интернете.
2. Контент-менеджеры не всегда ставят подходящие фотки.
3. Картинка ведь может и не загрузиться... по каким-то причинам.

Альтернативный текст должен быть человекочитаем. Он произносится скринридерами...

Формы тоже должны быть доступны для любого пользователя.