-
Notifications
You must be signed in to change notification settings - Fork 69
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
CMS Design Intent - UI/UX Feedback - VBA Regional Office Product #16073
Comments
Added Facilities team label per Slack thread, so this issue will show in our board, for prioritizing / addressing changes. FYI @xiongjaneg |
@BlakeOrgan - Follow-up question on this. Are we required/encouraged to show character counts for all text input fields? Plain text fields in Drupal are all limited to 255 characters (unless we indicate a shorter limit), but I don't think we've typically used the field widget with /counter for those inputs. Rich text fields are more often the scenario where we've been displaying character counts. Also, just noting the FE design system patterns for Max Length: |
@davidmpickett I was speaking more on rich text fields. Sorry about that! |
scheduled cross-product discussion for Dec. 11 |
Noted in #16068, "Some error messages are predetermined in Drupal. We may only need to write error messages for specific fields, but additional information will be needed from the CMS team to determine this." Also, feedback mentioned client-side validation, which isn't yet available from CMS Team. |
Noted in #16345, "Character counts are currently handled in a couple different ways" across Drupal. Standardization across Drupal is a larger question that will need CMS Team involvement. Facilities team can select and use one character count method across this product. |
For "Service Region name input field on the VA Service Regions page should have help text. Is there any standardized way that these should be named? If so, describe that." Service Region will be a dropdown menu vetted by stakeholders, not a text field. There will plain language instructions to select the region. |
@xiongjaneg - See below for response and let us know if you have any questions/concerns:
CMS team response: For error messages, these must be determined prior to launch (including any new error messaging written for specific fields). Past reviews from CMS team included only happy path, so ensure error handling has been internally reviewed by Facilities team. For client-side validation, understood this is not yet available, but would be great to include in designs and when these are triggered (implementation of client-side validation is NOT a blocker for launch).
CMS team response: @BlakeOrgan and @thejordanwood to have a working session regarding desired state. Can @thejordanwood schedule once she's ready to collaborate on this work?
CMS team response: Dropdown menus should still have help text. Use the CMS style-guide for guidance. cc: @BlakeOrgan |
@maortiz-27-80 Thanks for the follow-up. Summing up my understanding here. For Pilot MVP
Not a blocker
|
That's correct, @xiongjaneg. Let us know if you need anything else! cc: @BlakeOrgan |
@davidmpickett are any tasks remaining from this feedback? |
@mmiddaugh @Agile6MSkinner @jilladams Here's my read on the current state
|
I have done another review of Drupal and Figma and made note of any remaining instances of missing character counters throughout the VBA product to the body of the ticket. |
VFS actions
Feedback from CMS Team
The CMS Team practice areas will document their feedback on artifacts provided by VFS teams following the Must, Should, and Consider Framework. CMS Team reviewers may also provide additional notes that don’t comment on the artifacts themselves but are important for implementation (eg. engineering/coding notes or design system feedback).
Must
Should
Consider
Tracking Implementation
The text was updated successfully, but these errors were encountered: