MWPW-159427: Milo Catalog: Ipad specific issues

[mirafedas]

On iOS version 15 and lower there is a different way of managing the vertical stacking context, which puts the modal below the merch card. Adding z-index on the merch-sidenav resolves this problem.

Related Milo PR: adobecom/milo#2994

Resolves: MWPW-159427

Test URLs:

• Before: https://main--cc-–adobecom.hlx.live/products/catalog?martech=off
• After: https://mwpw-159427-ipad-issues --cc-–adobecom.hlx.live/products/catalog?martech=off (added a space to hide the link from PSI check)

For PSI check:
https://mwpw-159427-ipad-issues--milo--mirafedas.hlx.live/?martech=off&georouting=off

[aem-code-sync]

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

• 8efc302 ✅ (latest)
• 963f01a ✅
• a15639d ✅
• 84d0a60 ✅

[aem-code-sync]

	Page	Scores	Audits	Google
📱	/?martech=off&georouting=off
🖥️	/?martech=off&georouting=off

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[Axelcureno]

why is the only change is in minified file?

[mirafedas]

@Axelcureno every time when we make a change in merch-sidenav on Milo, we need to update the minified merch-sidenav in CC to keep it up to date. Here is the respective Milo PR: adobecom/milo#2994

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 422: MWPW-159427: Milo Catalog: Ipad specific issues due to missing verified label. kindly make sure that the PR has been verified

[yesil]

Can you clarify why merch-sidenav is still needed on cc w.r.t #412 (review) ?

[mirafedas]

Closing because the needed change will be added in the #412