fix #184: several styling and content issues

hlxsites / zemax

AEM Franklin Solution for https://www.zemax.com/

https://main--zemax--hlxsites.hlx.page/

Apache License 2.0

3 stars 1 forks source link

fix #184: several styling and content issues #192

Closed synox closed 1 year ago

synox commented 1 year ago

Fix #184

Test URLs:

Before: https://main--zemax--hlxsites.hlx.page/
After: https://184-style--zemax--hlxsites.hlx.page/

aem-code-sync[bot] commented 1 year ago

Hello, I'm Franklin 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[bot] commented 1 year ago

Page	Scores	Audits	Google
/

aem-code-sync[bot] commented 1 year ago

Page	Scores	Audits	Google
/
/fragments/subscribe-newsletter-fragment
/footer

kunwarsaluja commented 1 year ago

@synox can you review these linting and performance problems so LHS is 100

aem-code-sync[bot] commented 1 year ago

Page	Scores	Audits	Google
/

aem-code-sync[bot] commented 1 year ago

Page	Scores	Audits	Google
/

kunwarsaluja commented 1 year ago

@synox can you review #184 i think it's missing fixes for 3,4 from the list.