[MWPW-189040][DoodleBug] - Visibility of next carousel is seen more when compared to Figma

[drashti1712]

Resolves: MWPW-189040

Test URLs:

• Before: https://main--cc--adobecom.aem.live/?martech=off
• After: https://doodlebug-fix--cc--adobecom.aem.live/?martech=off

Dev validation

• Before:

• After:

[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

• 910604b ✅ (latest)
• 2040211 ✅
• f85b571 ✅
• fdf9d93 ✅
• 0301b5c ✅
• 5cf98b1 ❌
• 3d5b87d ❌
• 4e2bcdc ✅
• ea13951 ✅
• bde5aa7 ✅
• fabfa18 ✅

[aem-code-sync]

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

[milo-pr-merge-cc]

Skipped merging 891: [MWPW-189040][DoodleBug] - Visibility of next carousel is seen more when compared to Figma due to missing verified label. kindly make sure that the PR has been verified

[milo-pr-merge-cc]

Skipped merging 891: [MWPW-189040][DoodleBug] - Visibility of next carousel is seen more when compared to Figma due to missing verified label. kindly make sure that the PR has been verified

[nkthakur48]

@suhjainadobe Could you please review the PR?

[milo-pr-merge-cc]

Skipped merging 891: [MWPW-189040][DoodleBug] - Visibility of next carousel is seen more when compared to Figma due to missing verified label. kindly make sure that the PR has been verified

[SushmaNayak630]

Hi @drashti1712 , the given fix is working only in Samsung Galaxy S8+

issue still there in iphone and samsung s20


Could you please confirm on this

[nkthakur48]

@drashti1712 Could you please look at @SushmaNayak630's comment
CC: @suhjainadobe

[drashti1712]

Hi @SushmaNayak630
This is expected because as the screen width increases, the next slide would be more visible. According to figma, the carousel slide width is 300px and the gap is 16px between the slides. For devices with 360px screen width, the next slide would be seen only a little (according to figma). But for devices with screen width greater than 360px, the next slide would be seen more as the carousel width 300px and the gap 16 px remains fixed. This was also discussed with @spadmasa.

cc: @nkthakur48 @suhjainadobe

[spadmasa]

Validation on the pr in s8 as advised
#891
https://doodlebug-fix--cc--adobecom.aem.live/drafts/doodlebug/remove-object

[milo-pr-merge-cc]

Error merging 891: [MWPW-189040][DoodleBug] - Visibility of next carousel is seen more when compared to Figma Required status check "aem-psi-check" is expected.

[drashti1712]

Initially jira mentioned 'ipad' as well for the fix. However, after discussion with the team, it was confirmed to only apply changes for mobile. I had updated the breakpoint accordingly.

[SushmaNayak630]

Validation for https://jira.corp.adobe.com/browse/MWPW-189558
https://doodlebug-fix--cc--adobecom.aem.page/drafts/doodlebug/remove-object?milolibs=stage
As per drashtis confirmation , peek view will not come for 320 px

https://doodlebug-fix--cc--adobecom.aem.page/products/firefly/features/image-upscaler?milolibs=stage

https://doodlebug-fix--cc--adobecom.aem.page/products/firefly/features/ai-image-expander?milolibs=stage