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
282 stars 200 forks source link

Replace fontawesome icons in secondary nav #82075

Closed wesrowe closed 3 months ago

wesrowe commented 4 months ago

Description

User story

As a Cartography team member, I want to transition icons on the secondary nav to the new design system icons, so that the icons will stay the same after launch.

Notes

This ticket only covers icon updates in the secondary navigation component. A follow-on ticket will cover updating icons on the landing page (#83415).

Related conversations:

Tasks

Acceptance criteria

sterkenburgsara commented 4 months ago

Update 5/14:

sterkenburgsara commented 4 months ago

We now have our four icons decided for secondary nav, and developers can move forward with updating those. Those icons are as follows (all should be used at the smallest icon size of 24 px):

We still need to work out the two remaining icons that will go on the My HealtheVet landing page for Payments & Medical supplies sections.

cc @carlosfelix2 @jonathanjnelson

sterkenburgsara commented 4 months ago

Some options we've explored for icons under payments + medical supplies in Figma here

Will need to connect with auth experience team (and potentially other teams that may be affected if we want to consider alternative icons) to get their PoV.

wesrowe commented 4 months ago

I split the in-page icon updates into a new ticket

sterkenburgsara commented 4 months ago

Update on ticket about pulling in pill icon. Sounds like the DST will prioritize getting this into repos in their next sprint, which starts later this week. Wes is getting updates from DST team PM Carol Wong.

wesrowe commented 4 months ago

Hoping for an update tomorrow morning.

Note: we want to have the pill merged by Friday's go/no-go one way or another. So please move ahead with the work-around if it needs to be started to meet that timeline.

radavis commented 4 months ago

Here's the commit that adds the pill icon.

Once a new release (> v42.0.0) is made, we'll be able to update the component library package in vets-website and update icons.

radavis commented 3 months ago

component-library issue

When using the pill icon, there is a fill color value coded within the svg.

with fill without fill
Mobile Screenshot from 2024-05-28 11-05-59 Screenshot from 2024-05-28 11-06-29
radavis commented 3 months ago

DST is on it: https://github.com/department-of-veterans-affairs/vets-design-system-documentation/issues/2875

radavis commented 3 months ago

Update to component-library was released (v42.1.1). Unblocked

https://github.com/department-of-veterans-affairs/component-library/releases/tag/v42.1.1

allanto-ah commented 3 months ago

Environment: Staging

Verified that the pill icon for Medications is appearing as expected. All the other icons in the secondary nav are looking as expected as well

Screenshot 2024-06-04 at 11.51.48 AM.png Screenshot 2024-06-04 at 11.51.56 AM.png
brea11y commented 3 months ago

Verified that all icons (including the pill icon) have been swapped out with the new USWDS icons. Colors all match what is seen in Figma. No accessibility concerns.