Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

15632 CLP Aspect Ratio update #1791

Closed
wants to merge 2 commits into from
Closed

15632 CLP Aspect Ratio update #1791

wants to merge 2 commits into from

Conversation

randimays
Copy link
Contributor

@randimays randimays commented Nov 13, 2023

Summary

Campaign Landing Pages currently use a 7:2 aspect ratio that causes problems for rendering the image at various breakpoints.

Current design:

Screenshot 2023-11-13 at 4 13 23 PM

Design put together a new 1:1 design for Campaign Landing Pages so the text and the image would be side by side.

New design:

Screenshot 2023-11-13 at 4 12 36 PM

Related issue(s)

Testing done

Tested local build on:

  • /initiatives/protecting-veterans-from-fraud
  • /initiatives/veterans-experience-action-centers
  • /initiatives/national-buddy-check-week-talk-to-10-veterans
  • /initiatives/sign-in-securely-with-logingov
  • /initiatives/covid-flu
  • /initiatives/veteran-trust-in-va
  • /initiatives/end-of-life-benefits/
  • /initiatives/emergency-room-911-or-urgent-care
  • /initiatives/network-of-support
  • /initiatives/vote
  • /initiatives/recognizing-lgbtq-veterans-during-pride-month

Screenshots

Screenshot 2023-11-27 at 11 07 02 AM Screenshot 2023-11-27 at 11 06 45 AM Screenshot 2023-11-27 at 11 06 32 AM Screenshot 2023-11-27 at 11 06 18 AM Screenshot 2023-11-27 at 11 06 03 AM Screenshot 2023-11-27 at 11 05 49 AM Screenshot 2023-11-27 at 11 05 36 AM Screenshot 2023-11-27 at 11 05 23 AM Screenshot 2023-11-27 at 11 05 09 AM Screenshot 2023-11-27 at 11 04 30 AM

Acceptance criteria

  • CLP template(s) use the design-recommended image aspect ratio & updated gradient across all breakpoints where the image should appear
  • Design review is required, including verifying existing published CLPs in a Tugboat
  • Include alt text tag but ensure value is NULL ("") - verify with Laura in Tugboat
  • Mark PR DO NOT MERGE

@va-vfs-bot va-vfs-bot temporarily deployed to master/main/15632-clp-aspect-ratio November 13, 2023 22:33 Inactive
@randimays randimays force-pushed the 15632-clp-aspect-ratio branch from 95fc0fa to a636f85 Compare November 13, 2023 22:59
@va-vfs-bot va-vfs-bot temporarily deployed to master/main/15632-clp-aspect-ratio November 13, 2023 23:02 Inactive
@randimays randimays force-pushed the 15632-clp-aspect-ratio branch from a636f85 to 40f115f Compare November 14, 2023 15:27
@va-vfs-bot va-vfs-bot temporarily deployed to master/main/15632-clp-aspect-ratio November 14, 2023 15:32 Inactive
@randimays randimays force-pushed the 15632-clp-aspect-ratio branch 2 times, most recently from 9e01f6b to 9b40e2a Compare November 14, 2023 16:27
@va-vfs-bot va-vfs-bot temporarily deployed to master/main/15632-clp-aspect-ratio November 14, 2023 16:41 Inactive
@randimays randimays marked this pull request as ready for review November 14, 2023 17:45
@randimays randimays requested review from a team as code owners November 14, 2023 17:45
Copy link
Contributor

@chriskim2311 chriskim2311 left a comment

Choose a reason for hiding this comment

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

LGTM!

@randimays randimays force-pushed the 15632-clp-aspect-ratio branch from 9b40e2a to b9eb59d Compare November 20, 2023 15:46
@va-vfs-bot va-vfs-bot temporarily deployed to master/main/15632-clp-aspect-ratio November 20, 2023 16:17 Inactive
@randimays randimays force-pushed the 15632-clp-aspect-ratio branch from b9eb59d to e2934a7 Compare November 27, 2023 17:02
@va-vfs-bot va-vfs-bot temporarily deployed to master/main/15632-clp-aspect-ratio November 27, 2023 18:05 Inactive
@randimays
Copy link
Contributor Author

Closing due to coding sync issues with Tugboat. Will open another PR

@randimays randimays closed this Nov 28, 2023
@randimays randimays deleted the 15632-clp-aspect-ratio branch December 11, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants