Skip to content

Feature/actual vacancies#141

Open
badstandup wants to merge 22 commits intomasterfrom
feature/actual_vacancies
Open

Feature/actual vacancies#141
badstandup wants to merge 22 commits intomasterfrom
feature/actual_vacancies

Conversation

@badstandup
Copy link
Copy Markdown
Contributor

1)Карточки создаются из темплейта, все данные берутся из массива.

2)Добавлена пагинация, которая работает с данными из массива.

3)Реализован фильтр по заданными параметрам, если нужной карточки в массиве не оказалось, показывается, что такой вакансии не существует. Использовать фильтр можно бесконечно

4)Сделано на чистом JS.

Файлы скриптов необходимые для функционирования данной страницы:
*actual-vacancy-initial-data
*actual-vacancy-pagination
*actual-vacancy-render
*actual-vacancy-search

Страница actual-vacancies.html

Снимок экрана 2022-09-20 в 13 08 19

Снимок экрана 2022-09-20 в 13 09 04

AndreiSalnikov and others added 20 commits September 15, 2022 19:30
# Conflicts:
#	src/css/blocks/tender/tender.css
# Conflicts:
#	src/css/blocks/add-vacancy/__select/add-vacancy__select.css
#	src/css/index.css
#	src/index.html
@vercel
Copy link
Copy Markdown

vercel bot commented Sep 20, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
uvalery ✅ Ready (Inspect) Visit Preview Sep 20, 2022 at 10:58AM (UTC)

@badstandup badstandup linked an issue Sep 20, 2022 that may be closed by this pull request
@Anastacia-Tesli
Copy link
Copy Markdown
Contributor

Пока что есть конфликты с главной веткой🙈
Я пока что просто потыкала в пагинацию, работает, но при нажатии на любую страницу или вперёд/назад меня уносит на верх страницы
Так должно быть?🤔

# Conflicts:
#	src/css/index.css
#	src/index.html
#	src/pages/add-resume.html
#	src/pages/add-vacancy.html
#	src/pages/default.html
#	src/pages/profile-org.html
#	src/pages/vacancy-item.html
#	src/pages/vacancy.html
@AndreiSalnikov
Copy link
Copy Markdown
Contributor

@Anastacia-Tesli Привет, вроде как конфликты с главной веткой уже порешали. А с поднятием страницы так и должно быть, такая фича так сказать, просто не логично, что ты переключил страницу с вакансиями и смотришь снизу вверх

@Anastacia-Tesli
Copy link
Copy Markdown
Contributor

Ааа, да, действительно👍🏻 Я просто гляжу с мобильного, естественно без вопросов к верстке, но тк там фильтр расположен сверху, то улетает за пределы самого блока, практически к хедеру 🤔 а так наверное нормально
Я брала за исходник всего лишь три карточки на странице, их спокойно можно просматривать без поднятия. Но так вообще да, даже не подумала об этом :)

@badstandup
Copy link
Copy Markdown
Contributor Author

@Anastacia-Tesli мобильной версии пока нет, но в ней и так лучшее решение будет - убрать фильтр наверх

Снимок экрана 2022-09-20 в 13 48 25

@AndreiSalnikov
Copy link
Copy Markdown
Contributor

@Anastacia-Tesli если посмотрела, то можешь один аппрув нарисовать пожалуйста :)

@badstandup
Copy link
Copy Markdown
Contributor Author

@Anastacia-Tesli
kitty-smile

@Anastacia-Tesli
Copy link
Copy Markdown
Contributor

Я сейчас заметила, что на уровне разных страничек абсолютно разные ситуации со ссылками в хедере происходят )) но думаю это можно отдельно решить

@Anastacia-Tesli
Copy link
Copy Markdown
Contributor

Так, подождите. Сейчас на странице "Вакансии" слайдер (который тоже делали ребята) с похожими вакансиями. Куда тогда пойдет этот блок?
Я посмотрела задания для команд, в Актуальных вакансиях написано что Вакансии должны состоять из одного этого блока. А в Похожих вакансиях написано просто сделать слайдер на той же странице. И как быть @zuhijan

@AndreiSalnikov
Copy link
Copy Markdown
Contributor

@Anastacia-Tesli Хороший вопрос, мы просто только своё задание глянули, но мельком видели, что вроде туда уже кто-то свою страничку вставил, думали, что по ошибке. Ждём тогда ответа, что делать

@Anastacia-Tesli
Copy link
Copy Markdown
Contributor

Anastacia-Tesli commented Sep 20, 2022

По логике похожие вакансии должны располагаться на странице с вакансией конечно, под ней @CyrilLaz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Блок "Актуальные вакансии"

4 participants