Skip to content

Feature/profile org form validation#123

Open
Vova-Pol wants to merge 22 commits intomasterfrom
feature/profile-org-form-validation
Open

Feature/profile org form validation#123
Vova-Pol wants to merge 22 commits intomasterfrom
feature/profile-org-form-validation

Conversation

@Vova-Pol
Copy link
Copy Markdown
Contributor

@Vova-Pol Vova-Pol commented Sep 17, 2022

Добавлено:

  • скрипт валидации форм
  • верстка полей ошибок
  • стили ошибок
  • скрипт переключения состояния кнопки "Сохранить"
  • стили кнопки "Сохранить" в состоянии disabled

Исправлено:

  • атрибут type для полей ввода
  • удалены атрибуты pattern у полей ввода

@vercel
Copy link
Copy Markdown

vercel bot commented Sep 17, 2022 •
edited
Loading

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

Name Status Preview Updated
uvalery ✅ Ready (Inspect) Visit Preview Sep 19, 2022 at 1:50PM (UTC)

@talovikov
Copy link
Copy Markdown
Member

talovikov commented Sep 17, 2022

@Vova-Pol Привяжи задачу, пожалуйста :) Есть конфликты

@MariaMantusova
Merge branch 'master' into feature/profile-org-form-validation
c02ca3f 
# Conflicts:
#	src/css/blocks/profile-org/profile-org.css
#	src/index.js
#	src/pages/profile-org.html
@MariaMantusova MariaMantusova linked an issue Sep 17, 2022 that may be closed by this pull request
Валидация страницы "Профиль организации" #27
Open
@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 17, 2022

@Vova-Pol Привяжи задачу, пожалуйста :) Есть конфликты

Мы разрешили конфликты

CyrilLaz
CyrilLaz reviewed Sep 18, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@CyrilLaz CyrilLaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У вас при появлении текста ошибки в почти в каждое поле, блок ошибки двигает вниз всю форму, ведь возможно поправить... блок ошибки вниз отправить и размер текста уменьшить

@MariaMantusova
Merge branch 'master' into feature/profile-org-form-validation
c3bf528 
# Conflicts:
#	src/pages/profile-org.html
@Allxy
Copy link
Copy Markdown
Contributor

Allxy commented Sep 18, 2022

Ошибки некорректно отображаются и сдвигают другие элементы, так же они не красного цвета.
Посмотрите страницу /pages/add-vacancy.html
Нужно сделать так же как и там.

@Vova-Pol Vova-Pol requested review from Allxy and CyrilLaz September 19, 2022 09:59
@Allxy
Copy link
Copy Markdown
Contributor

Allxy commented Sep 19, 2022

Ссылка в хедере же не Блог должна называться а Профиль организации, судя по ТЗ

@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 19, 2022

А это не отдельная мини-задачка должна быть?

@Allxy
Copy link
Copy Markdown
Contributor

Allxy commented Sep 19, 2022

А это не отдельная мини-задачка должна быть?

Я это в вашем issue увидел

@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 19, 2022

Поняла, сейчас поменяю

@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 19, 2022

Поменяла)

@Allxy
Copy link
Copy Markdown
Contributor

Allxy commented Sep 19, 2022

Поменяла)

А остальные странички забыла?)

@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 19, 2022

Оу, я думала только на нашей, сейчас исправлю

@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 19, 2022

Теперь вроде точно все

@Allxy
Copy link
Copy Markdown
Contributor

Allxy commented Sep 19, 2022 •
edited
Loading

Теперь вроде точно все

index.html забыла

@MariaMantusova
Copy link
Copy Markdown
Contributor

MariaMantusova commented Sep 19, 2022

теперь все вроде

Allxy
Allxy reviewed Sep 19, 2022
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MariaMantusova MariaMantusova MariaMantusova left review comments

@Allxy Allxy Allxy approved these changes

@CyrilLaz CyrilLaz Awaiting requested review from CyrilLaz

At least 3 approving reviews are required to merge this pull request.

Assignees

@Vova-Pol Vova-Pol

@MariaMantusova MariaMantusova

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Валидация страницы "Профиль организации"

5 participants

@Vova-Pol @talovikov @MariaMantusova @Allxy @CyrilLaz