Floating images design updates #201

schooolman · 2024-03-29T16:51:58Z

Updating the styles for the floating-image block to match the new site design. Have also updated primary, secondary, and tertiary colors in the root styles.

Test URLs:

aem-code-sync · 2024-03-29T16:52:01Z

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

Re-run PSI checks
Re-sync branch

Commits

66fdc98 ✅ (latest)
475748a ✅
8cac776 ✅
4c9326f ✅
abfc95d ✅
e1508aa ✅

aem-code-sync · 2024-03-29T16:52:31Z

Page	Scores	Audits	Google
/

bstopp · 2024-04-01T14:01:50Z

blocks/floating-images/floating-images.css

@@ -43,14 +55,32 @@
  line-height: var(--line-height-m);
 }

-.block.floating-images p.button-container {
-  text-align: center;
+.block.floating-images p .button {


Can you check this and compare it to the use of p.button-container a and/or p.button-container a.primary throughout the site, if we can just update those, that'd be better.

Took a look and it seems like this is the main primary button style of the new design, so I moved those button styles to the main styles.css file.

bstopp · 2024-04-02T13:27:44Z

styles/styles.css

  display: inline-block;
-  padding: .75em 1.5em;
  font-family: var(--font-family-proxima);


Is this still Proxima?

I was planning on updating the fonts in a separate pull request, and then updating all --font-family-proxima to use the new font, and changing the variable name to --font-family-primary

I can do that in this PR if you would prefer though.