Skip to content

🐛Fix next carousel arrow showing up at the end.#25549

Merged
sparhami merged 2 commits intoampproject:masterfrom
sparhami:carousel-at-end
Nov 12, 2019
Merged

🐛Fix next carousel arrow showing up at the end.#25549
sparhami merged 2 commits intoampproject:masterfrom
sparhami:carousel-at-end

Conversation

@sparhami
Copy link
Copy Markdown

  • Change the isAtEnd method to work correctly when the carousel's
    width is not a whole number.

Fixes #25510

- Change the `isAtEnd` method to work correctly when the carousel's
width is not a whole number.
Copy link
Copy Markdown
Contributor

@wassgha wassgha left a comment

Choose a reason for hiding this comment

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

Nits

@sparhami sparhami merged commit f1a85ce into ampproject:master Nov 12, 2019
@sparhami sparhami deleted the carousel-at-end branch November 12, 2019 21:16
micajuine-ho pushed a commit to micajuine-ho/amphtml that referenced this pull request Dec 27, 2019
- Change the `isAtEnd` method to work correctly when the carousel's
width is not a whole number.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

With the <amp-carousel> 0.2 script, the right arrow sometimes shows on the last slide

4 participants