hlxsites / caesars

Caesars AEM Franklin Implementation
Apache License 2.0
1 stars 2 forks source link

Issue 48 mini forms #241

Open amol-anand opened 1 year ago

amol-anand commented 1 year ago

Fix #48

🔗 Test URLs:

📝 Description:

Driven by the type specified in the block and the query-index.json under /products. Buying tickets / making reservations are driven from the 'Secondary Url' in the product json. Some outstanding CSS finessing needed potentially since the original page was weird.

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.

aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/caesars-palace/drafts/amol/restaurants SI FCP LCP TBT CLS PSI
/caesars-palace/drafts/amol/shows SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/caesars-palace/drafts/amol/restaurants SI FCP LCP TBT CLS PSI
/caesars-palace/drafts/amol/shows SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/caesars-palace/drafts/amol/restaurants SI FCP LCP TBT CLS PSI
/caesars-palace/drafts/amol/shows SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/caesars-palace/drafts/amol/restaurants SI FCP LCP TBT CLS PSI
/caesars-palace/drafts/amol/shows SI FCP LCP TBT CLS PSI
tmathern commented 1 year ago

Only issue I see, but I don't know if it qualifies to prevent merging this, is that sometimes the dropdown is cut off:

Screenshot 2023-05-02 at 9 15 56 AM