| “Edit this page” | Paragon documentation site | @Adam Stankiewicz | [proposal] “Edit this page” on Paragon docs site to encourage contributions to the documentation from consumers.
Current state: “Leave feedback” Proposed state: “Leave feedback” | “Edit this page”
[decision] Yes, let’s do it. @Adam Stankiewicz to file issue and triage. |
1 minute | edge case for autosuggest | @Brian Smith | slack comment For a non-required field where freeform text is not allowed, if a user tabs out after entering freeform text should it be in an error state? My gut feeling is yes.
Notes: [Jeff] “If in error state, there should be a message explaining the nature of the error. Don't make them guess.” [Ben] Is this a search error? Input wasn’t valid, no matching found autosuggest result. [Jeff] Possible concern: User’s input is a valid, but the autosuggest item wasn't chosen. Possible validation message: “Failed to make a selection” If there’s only 1 option, force the selection. [Jeff] In this case, should we suggest a list of similar, matching options in this case?
[Matthew] Possible concern around making this a blanket decision for all cases depending on the context. [Ben] autosuggest is almost a form inside of a form, where there probably should be validation message. [Matthew] Do we have an example where this might come up? [Jeff] hard UX space, hard to nail down. [Jeff] @Brian Smith 's suggestion makes sense, but the “suggestion list” can be separate/deferred. [Brian] may not be best as part of Form.AutoSuggest itself. [Ben] maybe include a hook on blur for consumers to do something custom, per MFE.
Likely shouldn’t treat this as an actual validation error (like top-level error validation), blocking form validation. It’s more like a hint. Implications for aria-invalid Likely not necessary for non-required fields, but should have an appropriate label. [Adam] Form.Control.Feedback may not support a “hint” state that differs from isValid or isInvalid.
[Matthew] 3 options Alert pattern distinct from error messages Don't do anything, text vanish / blank out Block form submission and treat as a normal error.
[decision] We will treat as a normal error for now, and then defer/ticket out the “hint” pattern from a design perspective.
|
5 mins | SelectableBox.Set | @Max Frank | Adding aria-labelledby to the SelectableBox.Set in order to associate a label with the wrapping element [Jeff] Yes. Let’s make sure we update the component docs examples, too! [Adam] Consider prop spreading.
[decision] We will add support for this! @Max Frank to implement. |
