search

Netcentric / vg-macktrucks-com-rd

Franklin Site Redesign for https://www.macktrucks.com
Apache License 2.0
1 stars 12 forks source link

Tabbed carousel multiple instances get combined #109 #167

Closed Lakshmishri closed 1 year ago

Lakshmishri commented 1 year ago

Fix #109

Test URLs:

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

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.

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/drafts/lakshmi/v2/tabbed-carousel PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
TomaszDziezykNetcentric commented 1 year ago

@Lakshmishri Can we align the items on mobile to the align-items: flex-start;? Now it looks weird IMO

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/drafts/lakshmi/v2/tabbed-carousel PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
cogniSyb commented 1 year ago

@Lakshmishri Can we align the items on mobile to the align-items: flex-start;? Now it looks weird IMO

@TomaszDziezykNetcentric I’ve updated the branch together with two other changes. Could you check the PR?