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

Update VBA Facility post-review and collab cycle feedback #16327

Closed
28 of 43 tasks
xiongjaneg opened this issue Dec 6, 2023 · 12 comments
Closed
28 of 43 tasks

Update VBA Facility post-review and collab cycle feedback #16327

xiongjaneg opened this issue Dec 6, 2023 · 12 comments
Assignees
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team sitewide

Comments

@xiongjaneg
Copy link
Contributor

xiongjaneg commented Dec 6, 2023

Description or Additional Context

VBA Facility content type was reviewed by UX in #14888.
Collab cycle feedback was reveiwed by UX in 15989

Recommendations are documented in Figma.

Character counter and error states updates are covered under separate tickets.

Acceptance Criteria

  • Review Figma for updates and annotations. Consult Jordan as needed.
  • VBA Facility updates are made according to Figma.
    • The order of many fields has shifted from current design to new design
    • Text under Editing this Content has changed
    • Jump to edit renamed to On this page
    • Operating status fieldset
      • Is now part of Location and contact information fieldset
      • Text under operating status section says "This status will display on the facility's page and in VA's facility locator tool."
      • Help text under Operating status Details says "Add relevant details about how this status will impact Veterans and visitors."
    • Banner alert
    • Top of page information fieldset
      • Page introduction field is removed from content type
      • This fieldset is removed
    • Facility Image fieldset
      • Renamed from Facility photo
      • Is now part of Location and contact information fieldset
      • New text guidance added
    • Location and contact information fieldset
      • This fieldset now includes both editable (Operating Status & Facility photo) and non-editable fieldsets
      • This fieldset should now be a collapsable accordion
      • The order of fields in the non-editable fieldset has shifted
        • Benefit office hours may vary fetch field is now included here
        • Benefits hotline fetch field is now included here
        • Db data has been renamed Facility CMS data and has explanatory text
    • Prepare for your visit fieldset
      • This fieldset should now be a collapsable accordion
      • Initial text has changed
      • Accordion / Header help text has changed
    • Spotlight content fieldset
      • This fieldset should now be a collapsable accordion
      • Spotlight call to action link text help text should go to https://design.va.gov/content-style-guide/links#link-text
      • Why can't I edit this bubble on National spotlight should say "This content is standardized to provide consistent messaging for facilities nationwide."
    • Facility services = brand new tooltip
    • Other VA locations = brand new tooltip
  • Requires design review
  • Requires accessibility review

Team

Please check the team(s) that will do this work.

  • CMS Team
  • Public Websites
  • Facilities
  • Accelerated Publishing
@xiongjaneg xiongjaneg added the Needs refining Issue status label Dec 6, 2023
@xiongjaneg xiongjaneg added Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team labels Dec 6, 2023
@xiongjaneg xiongjaneg changed the title Update VBA Facility content type based on UX Review Update VBA Regional Office content types based on UX Review Dec 14, 2023
@xiongjaneg xiongjaneg changed the title Update VBA Regional Office content types based on UX Review Update VBA Regional Office content types for UX and CMS collab cycle reviews Jan 5, 2024
@xiongjaneg xiongjaneg changed the title Update VBA Regional Office content types for UX and CMS collab cycle reviews Update VBA Facility content type for UX and CMS collab cycle reviews Jan 23, 2024
@xiongjaneg xiongjaneg changed the title Update VBA Facility content type for UX and CMS collab cycle reviews Update VBA facility, service location, and centralized content post-review and collab cycle feedback Jan 23, 2024
@xiongjaneg xiongjaneg changed the title Update VBA facility, service location, and centralized content post-review and collab cycle feedback Update VBA Facility post-review and collab cycle feedback Jan 23, 2024
@xiongjaneg xiongjaneg removed the Needs refining Issue status label Jan 23, 2024
@davidmpickett
Copy link
Contributor

Current design VBA_Facility_Current.pdf

@davidmpickett
Copy link
Contributor

davidmpickett commented Jan 31, 2024

New design
VBA_Facility_new.pdf.zip

@davidmpickett
Copy link
Contributor

@omahane @swirtSJW @thejordanwood
I updated the ticket with a lot more details of the updates between the current and new design. I did not spell out every single detail, but this should be a pretty good guide

@laflannery
Copy link
Contributor

In the banner alert section there are 2 links in the help text that do not currently work, they are just refreshing the current page. Can these be updated?
image

@omahane
Copy link
Contributor

omahane commented Feb 6, 2024

@davidmpickett The changes to the design would change the header and content help text for all Accordions. I want to confirm that this is expected.

@omahane
Copy link
Contributor

omahane commented Feb 6, 2024

@davidmpickett The changes to the design would change the header and content help text for all Accordions. I want to confirm that this is expected.

In fact, there are a number of changes in the design that would affect the paragraphs wherever they appear. Is that desired?

@davidmpickett
Copy link
Contributor

@davidmpickett The changes to the design would change the header and content help text for all Accordions. I want to confirm that this is expected.

In fact, there are a number of changes in the design that would affect the paragraphs wherever they appear. Is that desired?

Yes. We iterated on the help text with that specifically in mind. It is possible that there may have been an instance or two that slipped through unintentionally. Once this is in PR, I can do a final review to confirm that we don’t have any unintended consequences

@omahane
Copy link
Contributor

omahane commented Feb 6, 2024

Spotlight call to action link help text

@davidmpickett
Screenshot 2024-02-06 at 2 03 47 PM

docroot/modules/contrib/linkit/src/Plugin/Field/FieldWidget/LinkitWidget.php
needs to change to allow for the use of field help text, which would be the ideal solution, but an issue needs to be made on the module to allow that.

@davidmpickett
Copy link
Contributor

@xiongjaneg @aklausmeier Here's a small spinoff ticket that would likely need to go to the CMS Platform team for resolution. Not likely a high-priority item, but wanted to make sure it was documented #17170

@laflannery
Copy link
Contributor

laflannery commented Feb 8, 2024

Noting that I changed the above ticket completely. It is a request to add a CSS property to the CMS design system as this should be a global change

@omahane
Copy link
Contributor

omahane commented Feb 9, 2024

I created that ticket for the module: https://www.drupal.org/project/linkit/issues/3420280

@davidmpickett
Copy link
Contributor

Spotlight call to action link help text

@davidmpickett Screenshot 2024-02-06 at 2 03 47 PM

docroot/modules/contrib/linkit/src/Plugin/Field/FieldWidget/LinkitWidget.php needs to change to allow for the use of field help text, which would be the ideal solution, but an issue needs to be made on the module to allow that.

@omahane here's a stub of an issue for bringing this to CMS Platform team #17201

@xiongjaneg Again, not likely a high priority, but just documenting for consideration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team sitewide
Projects
None yet
Development

No branches or pull requests

5 participants