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

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
283 stars 204 forks source link

[SW] Staging Review - Components or patterns don't align with guidelines, benefits (63674) (Post Launch) #64434

Closed Tom-Davis closed 1 year ago

Tom-Davis commented 1 year ago

During the Staging Review on 8/15/23 this post launch feedback was provided #63674

Platform Issue

Design components or patterns don't align with Design System guidelines.

Issue Details

On the disability compensation and pension benefits card, if the user clicks "Edit" they get a "Cancel" and "Update" button pair. The buttons have chevrons. In our design system these buttons do not have chevrons.

Link, screenshot or steps to recreate

VA.gov Experience Standard

Category Number 04, Issue Number 07

Other References

WCAG SC 3.2.4 AA

Platform Recommendation

Remove the chevrons within the buttons.

josephrlee commented 1 year ago

This looks like a pretty simple fix, I'm guessing it's a 1 pointer. Code to edit is here.

I think we just need to remove the iconLeft and iconRight lines.

      <ButtonGroup
        buttons={[
          {
            label: 'Cancel',
            onClick: onCancel,
            secondary: true,
            iconLeft: '«',
          },
          {
            label: 'Update',
            onClick: onUpdate,
            iconRight: '»',
          },
        ]}
      />
mgleo commented 1 year ago

Private Zenhub Image

OptionSelect commented 1 year ago

I have deleted the lines but am having trouble testing. Still working on this to prove fix worked.

HeatherRienks commented 1 year ago

@OptionSelect - any updates to share on this one?