hoka-hoka / ML.test

0 stars 0 forks source link

Pixel perfect, UX #16

Open iatsdotfatr opened 3 years ago

iatsdotfatr commented 3 years ago
  1. Привести демо-страницы к pixel-perfect.
  2. Проверить корректность работы всех подключенных плагинов, подключенных на страницах.
hoka-hoka commented 3 years ago

Готово.

iatsdotfatr commented 3 years ago
  1. Исправить логику изменения текста в инпутах, а также убрать обрезку текста сверху. Выбранные даты в календаре должны сразу появляться в инпутах, кнопка "Очистить" должна отменять выбор дат. Кнопка "Применить" закрывает календарь. Клик по экрану вне календаря также должен закрывать календарь. click outside

  2. Если блокируется скролл при открытии дропдаунов, то предварительно надо выполнить прокрутку таким образом, чтобы дропдаун полностью помещался в окно просмотра, и затемнять все окно просмотра, кроме дропдауна, иначе это очень неочевидное поведение. Проще вообще его убрать, в данном случае в нем особого смысла нет. accesibility

  3. Border должен менять цвет при переключении. border

  4. Цвет текста должен меняться при переключении. cut text, color toggle

  5. Маска не соответствует формату ММ ДД ГГГГ. incorrect year

  6. Нет кнопок, отображается "3 гостя", хотя ничего не выбрано, также обрезан текст. empty, bad font, clean, apply button

  7. Посмотреть в Best practices как использовать внешние ссылки, а ТАКЖЕ внутренние ссылки и внести соответствующие правки по всему проекту. links

  8. Подключить/написать плагин для рейтинга. rate plugin

  9. Screenshot from 2021-05-16 18-25-14 Screenshot from 2021-05-16 18-25-38 Screenshot from 2021-05-16 18-25-51

hoka-hoka commented 3 years ago

Исправил. Обрезание текста может остаться, т. к. у меня он отображается нормально и причину я не нашёл. Может это из-за того, что в разных ОС используются свои программы для обработки шрифтов.

image

iatsdotfatr commented 3 years ago
  1. Первая ссылка не выделяется жирным. Ссылки на соцсети сделать реальными, заглушки поставить на пустые ссылки, disable c них убрать. При наведении на ссылки менять их цвет, не только курсор. active link, disabled, no state

  2. Границы сильно расходятся с макетом. border

  3. Карточка тоже. Screenshot from 2021-06-03 11-56-05

  4. На большом разрешении карточки ломаются. nopp

  5. Также изменять курсор при наведении на текст. hover state

  6. Даты изначально не синхронизированы с календарем. Также неочевидное поведение, когда нажимаешь на первый дропдаун, а выбирается вторая дата, и для переключения на первую, надо на нее нажать. different dates Peek 2021-06-03 11-47

  7. На одних инпутах outline есть, на других нет. На странице Form Elements есть состояние фокуса для инпутов, его и используй. outlines

  8. Убрать предупреждение из консоли. remove warning

  9. При вводе в инпут, автоматически добавляются обратные кавычки (``). symbols

  10. Обрезание шрифтов есть только при инициализации, непонятно почему. Попробуй отсюда все нужные разрешения

hoka-hoka commented 3 years ago

Исправил.

iatsdotfatr commented 3 years ago
  1. Анимацию появления поправить или убрать, выглядит странно. no animation

  2. Открывать меню нажатием на всю иконку, а не на среднюю полоску, и менять курсор. access

  3. Нет фавиконки в продакшне. favicon

  4. Карточка слишком сильно сужается, 270px сделать минимальной шириной. И если делать шире, то картинку увеливать пропорционально. small proportional

  5. Эти ссылки типа внутренние, не надо их открывать в новой вкладке. Screenshot from 2021-06-21 12-12-21

  6. При уменьшении ширины экрана наложение элементов. Screenshot from 2021-06-21 12-29-25

  7. Здесь надо отступы добавить. offset Screenshot from 2021-06-21 12-29-37 Screenshot from 2021-06-21 12-30-59

  8. Когда дата не выбрана, ничего не должно выделяться, только текущая дата, зеленым градиентом. И диапазон выбора должен выделяться фиолетовым градиентом. selected selection

  9. На странице search room маска отображается неправильно. gggg

  10. Проверить работу всех страниц в Firefox. crossbrowser

hoka-hoka commented 3 years ago
  1. С календарём не всё понятно. Вот пользователь берёт раскрывает календарь, видит зелёную метку (текущая дата), а дальше ему приходится методом тыка по дням узнавать о наличии ещё и фиолетовых меток. Разве не лучше всё сразу отобразить? Если фиолетовые метки по умолчанию точно должны быть скрыты и появляться по очереди при клике по дням, то в каком именно порядке? Например, сначала появляется метка для неактивного, а потом активного (с которого происходит раскрытие календаря) поля даты или наоборот?
  2. По маске. Так в UI вообще не показано, как должен выглядеть placeholder для этого поля даты.
  3. Здесь исправил и теперь верхнее обрезание текста в input-полях должно пропасть, хоть у меня его и не было, но на твоих картинка вижу, что осталось.

Остальное исправил.

iatsdotfatr commented 3 years ago
  1. Фиолетовым отмечается выбранная дата, она же отображается в инпуте. Меньшая дата в левом, большая в правом. Можно или дать возможность выбирать обе даты при первом открытии календаря, тогда надо следить за тем, какая дата больше, какая меньше, и выводить ее в соответствующем инпуте. Или можно сначала при нажатии на любой инпут всегда открывать календарь, на котором выбирается первая дата, а для выбора второй даты надо явно нажать на второй инпут. Или просто установить плагин с готовым календарем и изменить его согласно макету.
  2. ДД.ММ - ДД.ММ
iatsdotfatr commented 3 years ago
  1. Здесь не исправлено при ширине экрана 320px, проще всего у контейнера установить боковые паддинги. no padding

  2. Здесь тоже наложение/вплотную. Screenshot from 2021-06-29 18-28-59

Screenshot from 2021-06-29 18-35-11

Screenshot from 2021-06-29 18-35-30

Screenshot from 2021-06-29 18-37-57

  1. Неправильные ссылки wrong link