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

Improve the way to distinguish optional from mandatory fields #829

Closed
sschuberth opened this issue Aug 16, 2024 · 0 comments · Fixed by #942
Closed

Improve the way to distinguish optional from mandatory fields #829

sschuberth opened this issue Aug 16, 2024 · 0 comments · Fixed by #942
Assignees
Labels
enhancement New feature or request ui Issues related to the UI.

Comments

@sschuberth
Copy link
Contributor

Whether a field is optional / mandatory should not be part of the field's description (like it currently is e.g. for an organization's description), but be an inherent UI part of the form itself.

Some random links on the matter:

Personally, I regard everything as mandatory in a form unless noted otherwise, so I'd go for only marking optional fields. Doing that as a dimmed suffix to the field's name, or as a dimmed text that's only shown as long as the field is empty seems to be to most reasonable to me.

@sschuberth sschuberth added enhancement New feature or request ui Issues related to the UI. labels Aug 16, 2024
@mnonnenmacher mnonnenmacher changed the title Improve the way to distingusi optional from mandatory fields Improve the way to distinguish optional from mandatory fields Aug 16, 2024
@Etsija Etsija self-assigned this Aug 30, 2024
Etsija added a commit to doubleopen-project/ort-server that referenced this issue Aug 30, 2024
Use an "(optional)" placeholder in all optional fields, and remove the
word "Optional" from the field's descriptions.

Resolves eclipse-apoapsis#829.

Signed-off-by: Jyrki Keisala <[email protected]>
@Etsija Etsija linked a pull request Aug 30, 2024 that will close this issue
github-merge-queue bot pushed a commit that referenced this issue Aug 30, 2024
Use an "(optional)" placeholder in all optional fields, and remove the
word "Optional" from the field's descriptions.

Resolves #829.

Signed-off-by: Jyrki Keisala <[email protected]>
github-merge-queue bot pushed a commit that referenced this issue Aug 30, 2024
Use an "(optional)" placeholder in all optional fields, and remove the
word "Optional" from the field's descriptions.

Resolves #829.

Signed-off-by: Jyrki Keisala <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ui Issues related to the UI.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants