Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

swapped out logo images on home page #397

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SMakaiTakori
Copy link
Contributor

@SMakaiTakori SMakaiTakori commented Mar 30, 2023

Changes

Swapped out the 3 home page logo images for cdhs, imatter, and co-services.

Screenshots

Screenshot 2023-03-30 163654

Checklist

  • [ ] if changes filter/search functionality, adds or updates unit tests
  • [ ] if adds a new page, adds accessibility tests
  • [ ] if adds a new page or new interaction, adds e2e test
  • [ ] if adds a new page or new interaction, sends google analytics events and updates GA doc
  • [ ] if changes filter functionality, updates filters doc

Fixes Replace 3 home page logos ticket

@SMakaiTakori SMakaiTakori added the release-minor Change that adds something new, but doesn't modify existing user flows or break existing functions label Mar 30, 2023
@SMakaiTakori SMakaiTakori requested review from a team as code owners March 30, 2023 20:37
@jeffmaher jeffmaher temporarily deployed to ownpath-pr-397 March 30, 2023 20:37 Inactive
Copy link
Contributor

@jlhogan jlhogan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🎉

Copy link

@alex-cds alex-cds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot_20230330-152055
Desktop LGTM; mobile spacing/layout needs to be tweaked so it doesn't overlap with any text!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-minor Change that adds something new, but doesn't modify existing user flows or break existing functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants