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

Review settings forms concerning browser validation attributes #400

Closed
mstilkerich opened this issue Sep 18, 2022 · 1 comment
Closed

Review settings forms concerning browser validation attributes #400

mstilkerich opened this issue Sep 18, 2022 · 1 comment
Milestone

Comments

@mstilkerich
Copy link
Owner

Revisit the UI settings forms for accounts and addressbooks and check where we can user browser builtin validation to improve usability, e.g.:

  • Specific input types may be available for some input (e.g., URL), which may allow browsers to do things as showing more appropriate keyboards on mobile devices
  • Mark required form fields as such so the browser can check on submission they have actually been filled
  • Placeholder text may be useful for some fields to indicate expected format
  • Title attribute could provide useful help, especially concerning available placeholders
  • Regex pattern can be specified for fields such as refresh time to enable the browser to validate the input
@mstilkerich mstilkerich added this to the 5.0.0 milestone Sep 18, 2022
@mstilkerich
Copy link
Owner Author

  • More specific input types not found. URL is not usable for the discovery URL as at least Safari expects a full URL including scheme, whereas we also want to accept a simple domain name here. For the time intervals, the time pickers are not usable because they allow to pick a time of day only (i.e. limited to 24 hours max)

  • Required form fields have been marked as such. This includes the name fields; although the backend does not care about empty account or addressbook names, from a user point of view it makes little sense to leave them empty.

  • Placeholder text has been added to indicate format of the time strings.

  • Regex patterns check that the time strings and domain name are in the expected format (the domain name pattern is very minimalistic and does not really check the validity of a URL).

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

No branches or pull requests

1 participant