Closed dzhaneta closed 1 year ago
У меня в основном по косметике:
- Немного не сходится позиционирование по ПиксПерф:
- Некоторые элементы отличаются по размеру/цвету:
- Не уверен, что это тут нужно:
- Думаю, будет хорошо, если все кликабельные компоненты, которые присутствуют, но пока не имеют функционала имели какую-то заглушку. Напр. выводили что-то в консоль или просто хотя бы нажимались без действия:
- Нет ховера у кликабельных элементов
- Что-то не то с иконкой "+" 🙃 Возможно лучше использовать готовую цветную из макета.
Привет! Спасибо за проделанную работу, всё хорошо, но есть несколько замечаний:
Все замечания выше, от Паши и Димы поддерживаю
У меня в основном по косметике:
- Немного не сходится позиционирование по ПиксПерф:
- Некоторые элементы отличаются по размеру/цвету:
- Не уверен, что это тут нужно:
- Думаю, будет хорошо, если все кликабельные компоненты, которые присутствуют, но пока не имеют функционала имели какую-то заглушку. Напр. выводили что-то в консоль или просто хотя бы нажимались без действия:
- Нет ховера у кликабельных элементов
- Что-то не то с иконкой "+" 🙃 Возможно лучше использовать готовую цветную из макета.
правки внесены в коммите fade38b
Привет! Спасибо за проделанную работу, всё хорошо, но есть несколько замечаний:
- Присоединяюсь к замечаниям Димы по pixelPerfect и БЭМ
- Заменить заголовок с h6 на h3 или h4
- Уже сейчас сделать, чтобы ссылкой на вакансию был не весь элемент. Чтобы клик по "..." не вёл на переход на вакансию, ссылка на календарь не вела на переход вакансию и т.д. (Надеюсь понятно объяснил)
- Последнюю пустую карточку (Новая вакансия) я бы тоже переместил в vacanciesList. (После того, как мы сделали Map в конец массива добавлял бы разметку для пустой карточки)
- Может быть исправить структуру: сейчас у нас идёт в который входят ссылки : Может быть карточку сделать изначально не как ссылку а как div или article, а внутри уже поместить ссылку? Или если оставить ссылки, то рассматривать это как навигацию и тогда ссылки поместить в nav?
все правки внесены при коммите fc0c1bf
как договорились, смерджила текущий вид для ревью параллельно решаю проблему с прокруткой и пропадающими тенями карточек
добавлен компонент VacancyCard, в VacanciesPage добавлена ссылка-карточка "Новая вакансия". Также добавлены несколько новых иконок и скоректированные временные входящие данные по вакансиям в папке temp. пришлось решать конфликты в lock файлах package.json и yarn - но вроде я там все правильно покрутила.