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

https://department-of-veterans-affairs.github.io/va-mobile-library/
ISC License
0 stars 0 forks source link

[Feature] Replace service emblems and VA SVGs with png. Add Space Force emblems. #327

Closed narin closed 1 month ago

narin commented 1 month ago

Description of Change

Replaces VA logo and service emblem SVGs with PNGs. Add Space Force emblems and VA logo with dark text.

Testing Packages

N/A

Screenshots/Video

N/A (See images in Files changed)

Testing

N/A

PR Checklist

Code reviewer validation:

Publish

If changes warrant a new version per the versioning guidelines and the PR is approved and ready to merge:

jessicawoodin commented 1 month ago

@narin and I discussed the issues and are leaning towards using the same files that VADS uses. For some reason, the app currently uses different emblems than VADS, and I’d prefer to be cautious and use the same emblems as VADS, even if it results in lower quality graphics. Currently, the files that are used in the app are not scaling properly when converted to PNG and exported at various sizes. By using the VADS PNGs, we’ll no longer have the scaling issues. I’m going to continue to think through this and will follow up with the files tomorrow.

TimRoe commented 1 month ago

For some reason, the app currently uses different emblems than VADS

The reason is because a React Native update broke using the PNG-as-SVG method being used for the service emblems, it had already been merged to develop, and backing out such a change would've been a big pain so SVG versions were found and swapped in in short order and ticket 5878 created to get actually good SVGs (which never happened because military branches were uncooperative). It was only much later realized that the VA Logo was still using PNG-as-SVG so maybe they could've been adjusted to continue working.

Currently, the files that are used in the app are not scaling properly when converted to PNG and exported at various sizes.

Not sure I follow this--since they're SVGs, they should scale fine (and looking at them in storybook seem to) and the PNGs in this ticket don't appear to be the SVGs currently in use.

Edit: Or I guess most of them could be the same with scaling screwed up--seems like that doesn't add up for the Marines/Navy though.

jessicawoodin commented 1 month ago

Going forward, the mobile library should provide the same emblems that VADS provides. I updated the Emblems.zip file in the ticket. The zip file also includes the two new Space Force logos. I will let VADS know about the two additional logos and also see if they might have higher quality files for all emblems.

TimRoe commented 1 month ago

Going forward, the mobile library should provide the same emblems that VADS provides. I updated the Emblems.zip file in the ticket. The zip file also includes the two new Space Force logos. I will let VADS know about the two additional logos and also see if they might have higher quality files for all emblems.

For clarity, VADS currently doesn't have the emblems at all. They are part of the VA.gov repo.

There is an outstanding ticket for VADS to incorporate them separately. Potentially that ticket could also serve for tracking getting higher quality versions (potentially SVGs of all reasonable files sizes that can scale without any issues for website, mobile, whatever).