search

MastersAcademy / frontend-course-2021

Репозиторій з курсу «Advanced Front-end» проекту Masters Academy
MIT License
5 stars 29 forks source link

4-modules #137

Closed kovalov closed 2 years ago

kovalov commented 2 years ago

https://kovalov.github.io/frontend-course-2021/homeworks/vladyslav.kovalov_kovalov/4-modules/

kovalov commented 2 years ago

Пару вопросов по реализации и вещах, которые хотел бы исправить/сделать правильно:

  1. Нормальная ли практика использовать input type="time"?
  2. Как лучше реализовать переключение часовых поясов? Моя реализация наслаивает текущий вариант и новый
  3. Какую заглушку лучше делать для поля? Стоит ли использовать input type="text" для вывода значений?
MZabolotnev commented 2 years ago

Пару вопросов по реализации и вещах, которые хотел бы исправить/сделать правильно:

  1. Нормальная ли практика использовать input type="time"?
  2. Как лучше реализовать переключение часовых поясов? Моя реализация наслаивает текущий вариант и новый
  3. Какую заглушку лучше делать для поля? Стоит ли использовать input type="text" для вывода значений?
  1. Тут больше подойдет type="date"
  2. Не совсем понял что ты вкладываешь в понятие "наслаивает"?
  3. Input для ввода текста, не для вывода.

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

kovalov commented 2 years ago

если покликать на разные селекты, то дата начинает моргать http://i.imgur.com/1UHyWLV.png смотриться как баг лучше дату другого пояса выводить рядом с текущим

Исправил

kovalov commented 2 years ago

@NickCashFlow Переписал shortestWeekDaysNumber. Прошу проверить и подсказать, как улучшить