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

Mark required fields in form #2334

Closed
BorghildSelle opened this issue May 28, 2024 · 5 comments
Closed

Mark required fields in form #2334

BorghildSelle opened this issue May 28, 2024 · 5 comments
Assignees
Labels
🍌 low hanging fruit 🚀 ready to deploy Use this if issue is ready to be deployed

Comments

@BorghildSelle
Copy link
Contributor

BorghildSelle commented May 28, 2024

Career contact form and possible all
Should have a line saying "All fields with * is required" and * at the end of each required fields label
Introduce meta text under label for describing country code and formats

Career fair and events form - phone with or without country code?

@millianapia
Copy link
Contributor

TODO:

  • Change text directly in sanity when we deploy

padms added a commit that referenced this issue Jul 1, 2024
@millianapia
Copy link
Contributor

when codefreeze is over, i will add * to required fields in sanity
Screenshot 2024-07-09 at 13 33 47

@meols
Copy link
Collaborator

meols commented Jul 15, 2024

Then we will see it in dev/stage first I assume? Should the explanatory text "All fields with* is mandatory" look more different from the form field texts (italic, smaller...)? The text should say "All fields with * are mandatory" (add a space in front of the asterix, and are instead of is)

@millianapia
Copy link
Contributor

@meols It is merged now, staging was reset to main. The text has now been rewritten, added another space after *, and made it smaller and in italics

Screenshot 2024-07-15 at 13 13 24

millianapia added a commit that referenced this issue Aug 6, 2024
* ✏️ Add mark required fields in form #2334

* ✏️ Add countrycode explanation field #2334

* ♻️ Moved to a different group #2334

* 📝 Style changes to mandatory info #2334

---------

Co-authored-by: Padmaja <[email protected]>
@millianapia
Copy link
Contributor

added * to all required fields in sanity

@millianapia millianapia added the 🚀 ready to deploy Use this if issue is ready to be deployed label Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍌 low hanging fruit 🚀 ready to deploy Use this if issue is ready to be deployed
Projects
None yet
Development

No branches or pull requests

4 participants