Skip to content

🐛 amp-story-shopping removes support for different multi-line border radii#37817

Merged
jshamble merged 6 commits intoampproject:mainfrom
jshamble:multiLineBorderRadius
Mar 15, 2022
Merged

🐛 amp-story-shopping removes support for different multi-line border radii#37817
jshamble merged 6 commits intoampproject:mainfrom
jshamble:multiLineBorderRadius

Conversation

@jshamble
Copy link
Copy Markdown
Contributor

@jshamble jshamble commented Mar 3, 2022

#closes #37808
Some animation CSS keyframes were not calculating the border-radius properly, this PR fixes them.

@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Mar 3, 2022

Hey @gmajoulet! These files were changed:

extensions/amp-story-shopping/0.1/amp-story-shopping-tag.css
extensions/amp-story-shopping/0.1/amp-story-shopping-tag.js

@jshamble jshamble requested a review from processprocess March 3, 2022 18:17
@jshamble jshamble changed the title 🐛 amp-story-shopping updated animation keyframes to support different multi-line border radius 🐛 amp-story-shopping updated animation keyframes to support different multi-line border radii Mar 4, 2022
@processprocess
Copy link
Copy Markdown
Contributor

I spoke with @hongcatlover and we decided to remove the different border radius on two line text.
#37808 is updated with details.

@processprocess
Copy link
Copy Markdown
Contributor

Nice. Please update the name of the PR to reflect the change. Once the linting check is resolved I think this will be good to go.

@jshamble jshamble changed the title 🐛 amp-story-shopping updated animation keyframes to support different multi-line border radii 🐛 amp-story-shopping updated animation keyframes to remove support for different multi-line border radii Mar 14, 2022
@jshamble jshamble changed the title 🐛 amp-story-shopping updated animation keyframes to remove support for different multi-line border radii 🐛 amp-story-shopping removes support for different multi-line border radii Mar 14, 2022
Copy link
Copy Markdown
Contributor

@processprocess processprocess left a comment

Choose a reason for hiding this comment

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

Awesome! (Also thank you for fixing my typo :)

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.

[amp story shopping] Remove multi line border radius code

4 participants