Skip to content

Conversation

@moglione
Copy link

This commit adds a "Build and Development" section to the README.md file. This section provides clear instructions on how to install dependencies, build the project, and run the development server. It also clarifies that the project uses a custom build system.

This commit adds a "Build and Development" section to the README.md file. This section provides clear instructions on how to install dependencies, build the project, and run the development server. It also clarifies that the project uses a custom build system.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

This commit adds a new file, amp-story-player-instructions.md, which contains instructions on how to build and run the amp-story-player component in isolation. This helps developers who want to work on this specific component without building the entire project.
This commit implements a vertical swipe navigation for the amp-story-player component, similar to TikTok. The gesture handling logic has been updated to detect vertical swipes, and the CSS animations have been adjusted to create a smooth vertical transition between stories.

This also includes documentation on how to build and serve the component in isolation.
This commit refactors the amp-story-player component to use a vertical swipe navigation, similar to TikTok.

Key changes:
- The gesture handling logic in amp-story-player-impl.js has been updated to detect vertical swipes (swipe up to advance, swipe down to go back).
- The CSS animations in amp-story-player-shadow.css have been adjusted to create a smooth, fluid vertical transition between stories.
- Mobile compatibility has been ensured by updating the iframe capabilities to explicitly allow vertical swipes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants