antonymofyou / HireEmployeesVue

Репозиторий - система найма для Vue
0 stars 1 forks source link

feat: manager list on VacancyEditPage #62

Open IvanMalik0 opened 2 months ago

IvanMalik0 commented 2 months ago

Добавить компонент, отображающий менеджеров вакансии в редактировании вакансии.

antonymofyou commented 1 month ago

вы кажется передали мне неработающий ПР 62. Пишу замечания:

  1. Добавление менеджеров не работает
  2. Менеджеры(уже добавленные) у вакансии не отображаются
  3. В попапе при добавлении селект надо выровнять посередине
  4. Для менеджера надо создать отдельный общий компонент, как StatusColored, но для менеджеров
antonymofyou commented 1 month ago

Правки по ПР:

  1. Шрифт у менеджеров сделать на пару единиц меньше
  2. margin-top убираем
  3. gap делаем 10 пикс
  4. при удалении статуса делаем попап
  5. выпадающий список менеджеров делаем только при нажатии на плюсик. И выводим в попап.
  6. Возле плюсика надпись Добавить не нужна
  7. при уалении менеджера делаем попап подтверждения
  8. радиус делаем 20пкс
  9. Список Менеджеров - меняем на Менеджеры вакансии
  10. у блока менеджера padding: 7px 13px; делаем таким
  11. менеджеров вакансии помещаем над статусами
  12. max-height: 35px; надо убрать. иначе при переносе внутри - некрасиво
  13. Как я говорил - вывести блок с менеджером - в отдельный (общий!) компонент
Danila-Sergeev commented 1 month ago

Исправил по ПР:

+1.Шрифт у менеджеров сделать на пару единиц меньше +2.margin-top убираем +3.gap делаем 10 пикс +4.при удалении статуса делаем попап +5.выпадающий список менеджеров делаем только при нажатии на плюсик. И выводим в попап. +6.Возле плюсика надпись Добавить не нужна +7.при удалении менеджера делаем попап подтверждения +8.радиус делаем 20пкс +9.Список Менеджеров - меняем на Менеджеры вакансии +10.у блока менеджера padding: 7px 13px; делаем таким +11.менеджеров вакансии помещаем над статусами +12.max-height: 35px; надо убрать. иначе при переносе внутри - некрасиво +13.Как я говорил - вывести блок с менеджером - в отдельный (общий!) компонент

antonymofyou commented 1 month ago
  1. Добавление менеджера -> Добавить менеджера вакансии
  2. По неймингу. У общего компонента лучше не ManagerEdit, а что-то более общее с менеджером связанное сделать
  3. Кнопка удаления. Посмотри как у статусов вакансии. Кликаем - и у нас появляется крестик удаления. Нажимая на него - делаем попап подтверждения удаления
  4. в вакансии 64 не получается удалить Давида Быстрова. Нужно выяснить почему
  5. Когда закрываем попап с ошибкой, потом снова открыаем попап удаления - ошибка в попапе остается. А должен быть новый попап без ошибки.
  6. Ошибка должна писаться только в попапе, когда нажали - подтвердить удаление. У тебя она сейчас пишется и под плюсиком.
  7. Плюсик сделать не под менеджерами, а сразу после последнего менеджера, в том же ряду
antonymofyou commented 1 month ago
  1. Почему при нажатии на одного менеджера - крестик показывается у всех?
  2. При нажатии вне поля менеджера крестик не убирается
Danila-Sergeev commented 1 month ago

исправил: Почему при нажатии на одного менеджера - крестик показывается у всех? При нажатии вне поля менеджера крестик не убирается

antonymofyou commented 1 month ago

Вынести дизайн в общий компонент