department-of-veterans-affairs / va-mobile-app

"If VA were a company, it would have a flagship mobile app."
https://department-of-veterans-affairs.github.io/va-mobile-app/
17 stars 2 forks source link

Update logs and emblems global implementation #9632

Closed ajsarkar28 closed 1 month ago

ajsarkar28 commented 1 month ago

Monthly Update for Stakeholders

Date Progress & Key Accomplishments Issues/Risks/Blockers
9/9/24 Not started
9/17/24 Engineering in progress

Overview

What was updated in the Design System?

In va-mobile-library, the official VA logos and emblems were updated in version 0.6.0 of the Assets package and Figma library to be consistent with VA.gov. The new files are:

What changes are recommended in the Flagship Mobile App?

  1. In va-mobile-app, the VA logos and emblems in the VAIcon component should be updated as follows. After the files are updated, the vaParentLogo and dodBranch files should be removed from the VAIcon component.
    • Update: Logo.svg :arrow_right: VAOnDark.png
    • Update: AirForce.svg :arrow_right: vic-air-force-coat-of-arms.png
    • Update: Army.svg :arrow_right: vic-army-symbol.png
    • Update: CoastGuard.svg :arrow_right: vic-cg-emblem.png
    • Update: MarineCorps.svg :arrow_right: vic-usmc-emblem.png
    • Update: Navy.svg :arrow_right: vic-navy-emblem.png
    • New: VAOnLight.png
    • New: vic-space-force-logo-on-dark.png
    • New: vic-space-force-logo-on-light.png
    • New: vic-space-force-logo.png
  2. In Figma, the VA logos and emblems should be updated in the shipped files. After the files are updated, the Logo component should be removed from the Flagship Library file.

Where are these files used in the Flagship Mobile App?

Currently, the logos and service emblems are used on the home screen, profile screen, and Veteran Status.

Additional resources

Acceptance criteria

TKDickson commented 1 month ago

@theodur can you associate your PR with this ticket, instead of the epic itself? I will need to move my test plan over here as well, note to self

TKDickson commented 1 month ago

VA icons show appropriately, light and dark mode, on:

Military branch icons show appropriately, light and dark mode, on:

Also:

TKDickson commented 1 month ago

@theodur on the veteran status screen (which doesn't have light and dark mode variations), previously we were always showing the (equivalent of the) vic-space-force-logo-on-dark.png logo, so the "USSF" text would have sufficient contrast.

On this branch/with this change, the space force logo on the veteran status screen is changing with dark/light mode, which is causing this contrast issue (green circle mine/added): image.png

Can we go back to having an override for light mode, to keep the sufficient contrast, even with the DS changes?

TKDickson commented 1 month ago

And if all this ^ is actually something to chat about with the design system team, let me know!

theodur commented 1 month ago

@TKDickson Good catch, that was because the new MilitaryBranchEmblem component defaulted to setting it based on the theme. I added an override option to it, so this should be fixed now

TKDickson commented 1 month ago

Space Force emblem now displaying with enough contrast, light and dark mode, regardless of screen.

TKDickson commented 1 month ago

Running iOS and Android scripts (something appears to have gone awry with the PR, because they're not running there automatically)

TKDickson commented 1 month ago

@theodur looks like you need to make changes to the VeteranStatus detox script? Failing on emblems checks

theodur commented 1 month ago

Dang it, thought I got them all. Should be fixed now @TKDickson. I'll wait and see if the PR decides to automatically kick off the E2E tests. If not, I'll manually run them

theodur commented 1 month ago

No dice, had to run them manually:

TKDickson commented 1 month ago

Sadly, the VeteranStatusCard e2e script is also still failing due to logo/emblem change.

theodur commented 1 month ago

Fixed it and kicked off new test runs

https://github.com/department-of-veterans-affairs/va-mobile-app/actions/runs/11129832012

https://github.com/department-of-veterans-affairs/va-mobile-app/actions/runs/11129848124

TKDickson commented 1 month ago

Whoops, I accidentally cancelled your runs. But mine passed (or at least the failures aren't this branch)! iOS and Android