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

Section metadata options: background colour & image #64 #93

Merged
merged 17 commits into from
Oct 19, 2023

Conversation

kesiah
Copy link
Collaborator

@kesiah kesiah commented Sep 27, 2023

Fix #64

Test URLs:

Section metadata can contain the following styles:

  • black background
  • gray background
  • gray background overlap (used for testimonial block)
  • background with dots (used for solutions + cards; can be used in combination with gray background)

The section metadata can contain a background image, for columns info variant. See https://64-section-bg--vg-macktrucks-com-rd--netcentric.hlx.page/drafts/syb/columns and how it’s done in Franklin:
Screenshot 2023-10-17 at 11 25 42

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 27, 2023

Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-run PSI Checks

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 27, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 27, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 27, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 27, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 28, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Sep 29, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 9, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 9, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 16, 2023

Page Scores Audits Google
/drafts/laura/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@markovukiceviccn
Copy link
Collaborator

markovukiceviccn commented Oct 16, 2023

Screenshot 2023-10-16 at 18 56 10

On this link https://64-section-bg--vg-macktrucks-com-rd--netcentric.hlx.page/drafts/laura/pdp
Heading is missing word-wrap property

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 17, 2023

Page Scores Audits Google
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/syb/columns PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/homepage PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 17, 2023

Page Scores Audits Google
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/syb/columns PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/homepage PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 17, 2023

Page Scores Audits Google
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/syb/columns PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/homepage PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@cogniSyb cogniSyb changed the title 64 section bg Section metadata options: background colour & image (#64) Oct 17, 2023
@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 17, 2023

Page Scores Audits Google
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/syb/columns PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/homepage PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI


.section--background-with-dots::after {
background-image: url('../../media/dots.svg');
background-position: top right 10px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shorthand possible: background: url('../../media/dots.svg') top right 10px;

@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Oct 19, 2023

Page Scores Audits Google
/drafts/laura/pdp PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/drafts/syb/columns PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
/homepage PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@cogniSyb cogniSyb merged commit a046665 into develop Oct 19, 2023
2 checks passed
cogniSyb added a commit that referenced this pull request Nov 7, 2023
cogniSyb added a commit that referenced this pull request Nov 7, 2023
cogniSyb added a commit that referenced this pull request Nov 7, 2023
@cogniSyb cogniSyb added the FR Functional requirement label Nov 7, 2023
@cogniSyb cogniSyb changed the title Section metadata options: background colour & image (#64) Section metadata options: background colour & image #64 Nov 7, 2023
@WendyKruger WendyKruger removed this from the MT Solutions milestone Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FR Functional requirement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants