ampproject / ampproject/amphtml

[amp story player] prev and next story buttons display on desktop full bleed stories

Open
#36,263 1 comment 0 reactions 1 assignee Claimed by @processprocess View on GitHub
P1: High Priority Stale Type: Bug WG: stories
Dominant language
JavaScript
Stars
14.9k
Forks
4.1k
PR merge metrics
No merged PRs in 30d

Description

### Description

These should not be visible in the player when the story is desktop-bull bleed mode.

This function in the player may be a good place to start:
[onUiStateUpdate_](https://github.com/ampproject/amphtml/blob/main/src/amp-story-player/amp-story-player-impl.js#L974-L982)

[css code pointer](https://github.com/ampproject/amphtml/blob/main/css/amp-story-player-shadow.css#L108-L128)

![image](https://user-images.githubusercontent.com/3860311/136249717-256f16c5-0bed-4646-a723-a2361194227d.png)

### Reproduction Steps

Open a player with a full bleed story on desktop

### Relevant Logs

_No response_

### Browser(s) Affected

_No response_

### OS(s) Affected

_No response_

### Device(s) Affected

_No response_

### AMP Version Affected

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.