Skip to content

[MWPW-189973]Fix transition screen image issue on iphone#699

Merged
sanjayms01 merged 2 commits intostagefrom
iphone-image
Mar 13, 2026
Merged

[MWPW-189973]Fix transition screen image issue on iphone#699
sanjayms01 merged 2 commits intostagefrom
iphone-image

Conversation

@Ruchika4
Copy link
Collaborator

@Ruchika4 Ruchika4 commented Mar 12, 2026

  • fix transition screen image issue on iphone safari
  • This issue was only happening when video autoplay was set at once

Resolves: MWPW-189973

Test URLs:
Firefly

Photoshop:

Acrobat:

Testing note:
Please test across browser and devices.
Please test across workflows to make sure it works fine for when video is set to autoplay once(firefly) and when set to autoplay in loop (acrobat and photoshop)

@aem-code-sync
Copy link

aem-code-sync bot commented Mar 12, 2026

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

@Ruchika4 Ruchika4 changed the title Iphone image [MWPW-189973]Fix transition screen image issue on iphone Mar 12, 2026
video.play()?.catch(() => {});
}
};
if (video.readyState >= 3) playWhenReady();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ruchika4 Just for my understanding, what does this mean?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @sanjayms01 video.readyState is a read-only property on the DOM HTMLVideoElement. video.readyState >= 3 means the browser has enough data that playback can start.

@Ruchika4 Ruchika4 requested a review from JingleH March 13, 2026 18:02
@asonnalagi
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants