Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Отредактировать текст и поля, добавить пару фич #711

Open
Alisa-V-H opened this issue Jan 16, 2024 · 11 comments · May be fixed by #744
Open

Отредактировать текст и поля, добавить пару фич #711

Alisa-V-H opened this issue Jan 16, 2024 · 11 comments · May be fixed by #744
Assignees
Labels

Comments

@Alisa-V-H
Copy link

Alisa-V-H commented Jan 16, 2024

Problem description

Недавно мы меняли поля в резюме Хекслет CV, и там вспылила небольшие недоработки, которые нужно исправить.

  1. Опечатка в названии поля "Описание проектов"
  2. Навыки смотрятся большим полотном, если студентов вписывает сразу много навыков и оформляет их столбиком. Это неудобно читать ни карьерному консультанту, ни работодателю, ещё и смотрится некрасиво
  3. Нет поля для краткого описания компании. Нужно обязательно указывать, чем занималась компания, в которой работал студент, это упрощает решение работодателя по нему. Ребята забывают указывать эту информацию, несмотря на то, что в КТ есть чёткая рекомендация.

Proposed solution

Исправить опечатку в названии поля "Описание проектов":
image

Ограничить количество навыков до 10 и оформить их в 2 столбика, по 5 навыков в каждом столбике:
image

Добавить поле "Описание компании". Ограничить описание 150 символов, чтобы не писали полотна.
Описание компании добавить между "Компания" и "Должность"
image

@eKulshan
Copy link

взял в работу

@eKulshan
Copy link

@Alisa-V-H Добрый день!
Уже было исправление ранее. Какой в итоге правильный вариант? Описания или Описание?
Судя по тому что это инпут под множественные проекты, то все таки описания?

@fey
Copy link
Collaborator

fey commented Apr 10, 2024

@Alisa-V-H

Навыки смотрятся большим полотном, если студентов вписывает сразу много навыков и оформляет их столбиком. Это неудобно читать ни карьерному консультанту, ни работодателю, ещё и смотрится некрасиво

Тут надо определиться, какой формат описания навыков должен быть, т.е. его надо ограничить. Сейчас там можно написать как угодно и что угодно. Т.е. просто взять и сделать два столбика не получится - пользователь может написать их и не в виде списка, или большим текстом.
image
https://cv.hexlet.io/ru/resumes/3803
image
https://cv.hexlet.io/ru/resumes/3806
image

image
image
image
image

@Alisa-V-H
Copy link
Author

Мы можем ограничить количество символов, чтобы не получалось полотен?

@eKulshan
Copy link

Это просто текстовое поле, так что, да, ограничить можно - вопрос насколько?
Но поскольку мы не контролируем структуру внутри (markdown), все равно можно получить нечитаемое полотно даже в ограниченном количестве символов или столбик навыков высотой в страницу.

Возможно стоит отталкиваться от желаемого "идеального" вида и уже смотреть насколько мы можем привести текущую реализацию к этому виду?

@Alisa-V-H
Copy link
Author

Желаемый вид, это два столбика, в каждом по 5 навыков -- не больше.

@fey
Copy link
Collaborator

fey commented Apr 15, 2024

пока задачка не готова к взятию в работу - надо обсудить с продукт овнером

@eKulshan
Copy link

Отпишите тут пожалуйста, после уточнения тз, я закончу таску

@fey
Copy link
Collaborator

fey commented Apr 25, 2024

@eKulshan
Один скилл-одна строка. Дальше парсим построчно и выводим в 2 столбца по 5 элементов.

@fey
Copy link
Collaborator

fey commented Apr 27, 2024

@eKulshan можно брать в работу. если не получиться за недельку, то я попрошу ребят из разработки запилить. Так что жду вас пр!

@fey fey moved this to 🔖 Ready in Hexlet-CV Apr 27, 2024
@eKulshan
Copy link

eKulshan commented May 6, 2024

Приступил

@eKulshan eKulshan linked a pull request May 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🔖 Ready
Development

Successfully merging a pull request may close this issue.

3 participants