department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
97 stars 69 forks source link

VBA RO preview observations to address before launch #16901

Closed xiongjaneg closed 5 months ago

xiongjaneg commented 8 months ago

Description

This is a placeholder ticket to capture questions or comments about VBA RO pilot MVP before launch. Subsequent tickets may need to be stubbed from these observations.

Preview observations, resolved as of 3.25.2024

Screenshots

Screenshots of the VA benefits hotline number ![image](https://github.com/department-of-veterans-affairs/va.gov-cms/assets/55411834/ad21b4a5-7458-41a5-b42a-5d69320d50e7)
Get updates screenshots ![image](https://github.com/department-of-veterans-affairs/va.gov-cms/assets/55411834/f9da2807-bc08-4062-b85f-f9c1f8e1ce71)
mmiddaugh commented 8 months ago

~Not sure if the "Other nearby VA locations" part has been wired yet - the heading renders but no locations are included.~ resolved as of 3.25.2024

Screenshot ![image](https://github.com/department-of-veterans-affairs/va.gov-cms/assets/55411834/43f92f27-f42f-42d4-9528-f64fb77cc4a5)
mmiddaugh commented 8 months ago

I'm not sure this help text is entirely meaningful but it could benefit from a space between to and local in any case. (It may make sense to address this as part of Review error states recommendations and feasibility for VBA content types #16906)

Screenshot image.png
mmiddaugh commented 8 months ago

@laflannery - Does "Can't find the service you're looking for?" have the correct heading level? I did not expect it to have a jumplink - it seems out of place to assume at the top of the page that I can't find what I am looking for.... Perhaps it's because the Services header and service accordions aren't yet present in this preview? Preview staging link

Screenshot image.png
laflannery commented 8 months ago

@mmiddaugh So it is but I don't disagree with you about the anchor link, here's my problem:

I originally, based on what I was seeing from Sketch, said this should be an H3. Which meant it would NOT show in the "On this page" list. My reasoning was that it was a child of "Other services" which made sense to me:

Screenshot 2024-01-19 at 11 55 02 AM

However, I did just ask this to be updated to an H2 because the logic I thought was built into this block was not what I thought. This block just shows if it's added into the CMS and when an example page was created I saw an instance that didn't make sense to me - where "Can't find the service you're looking for?" was a child of "Veteran Benefits".

Screenshot 2024-01-19 at 12 02 29 PM

To me those two things felt more like siblings, not parent and child, especially if this can show anywhere under any accordion section. That was why I asked for this to change to an H2, the consequence of that is now it's in the "On this page".

Questions:

Let me know if you want to sync on this because I realize that was a lot of info.

mmiddaugh commented 7 months ago

@laflannery and I are on the same page about "Can't find what you're looking for"

The good news is that it will can serve Veterans and other beneficiaries who need additional help, whether they realize they need it at the top or bottom of the page.

davidmpickett commented 5 months ago

I'm not sure this help text is entirely meaningful but it could benefit from a space between to and local in any case. (It may make sense to address this as part of Review error states recommendations and feasibility for VBA content types #16906)

Screenshot

We are currently unable to change the interface text in this section. This is a CMS Team issue (#16452) that would need to be resolved.

The issue with the spacing seems to be a regression. "to" and the following label are actually entirely separate elements. If you highlight them, you can see there are two overlapping highlights.

Screenshot 2024-03-27 at 8 25 11 PM

They were properly spaced back in December as seen in my screenshot on the linked issue.

290887746-0fc02e7e-4be1-406c-8381-2899164dfeaa

This regression is not specific to Facilities, as it can be seen in on similar interfaces on Public Websites. For instance, Resources and Support Detail page content type.

Screenshot 2024-03-27 at 8 26 35 PM

Screenshot 2024-03-27 at 8 34 14 PM

mmiddaugh commented 5 months ago

see also: "Additional Hours Options" shows as Italics on VBA #17686

jilladams commented 5 months ago

I'm looking through VBA stuff and came across this ticket for the first time. Can y'all confirm that the intent here is that any feedback out of Editor training sessions comes here for review / to be ticketed?

If so, I think Dave's comment 2 up is a dangler with no ticket? https://github.com/department-of-veterans-affairs/va.gov-cms/issues/16901#issuecomment-2024244708

In the spirit of simplifying onboarding / expectations for a new PM, how would you feel about us closing this and just cutting new tickets for anything identified during sessions? (@laflannery @mmiddaugh )

laflannery commented 5 months ago

I'm not actually sure what the original intention of this ticket was here tbh. I personally agree with your idea of just making individual tickets though, I think that's easier than a group ticket approach like this.

davidmpickett commented 5 months ago

Spun out my comment into a separate ticket so this can close https://github.com/department-of-veterans-affairs/va.gov-cms/issues/17831