-
Notifications
You must be signed in to change notification settings - Fork 177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Editor and output: Custom page advancement #12318
Conversation
Plugin builds for bef41f0 are ready 🛎️!
|
Size Change: +739 B (0%) Total Size: 2.72 MB
ℹ️ View Unchanged
|
packages/story-editor/src/components/panels/shared/generalPageAdvancement.js
Outdated
Show resolved
Hide resolved
# Conflicts: # includes/Experiments.php
Verified this feature via web-story: Custom page advancement#12318 Status:
Scenario: Settings and Defaults
UX Suggestion |
Scenario: Overriding and Duration value
|
Scenario: Using the page adv settings in Preview & Publish
|
Scenario: Corner Cases and Negative Testing
Example - Page Adv Duration is 6 seconds, but embedded video is of 11 sec. Then the page will remain open for 11 sec till video auto plays to completion. However, if the Loop option is turned on for the video, then the page remains open for the Duration of 6 seconds. @swissspidy -- is it valid to have the page adv auto duration ignored if there is a video playing in the page that is longer duration ? |
Yes this is expected behavior. |
Thanks for thinking about this and for the suggestion! I believe @aaskedall recommended using something like "Default Page Advancement" in the Document settings, we're currently still waiting for the mocks though. |
Context
This adds a new custom per-page page advancement panel.
User-facing changes
Testing Instructions
This PR can be tested by following these steps:
Checklist
Type: XYZ
label to the PRFixes #12210