Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 [Story interactive] Fix Container Rendering Issue for Image Quizzes and Polls #35635

Merged
merged 6 commits into from
Aug 12, 2021

Conversation

Brandons42
Copy link
Contributor

Fixed blue pixels shown in below screenshots that are visible when components are placed on white background

image

image

After this PR:

Screen Shot 2021-08-11 at 7 23 08 PM

/cc @mszylkowski
/cc @processprocess

@amp-owners-bot amp-owners-bot bot requested a review from Enriqe August 12, 2021 05:28
@amp-owners-bot
Copy link

Hey @gmajoulet, @mszylkowski! These files were changed:

extensions/amp-story-interactive/0.1/amp-story-interactive-img.css

@@ -19,7 +19,7 @@
--i-amphtml-interactive-img-option-gap: 1em !important;
--i-amphtml-interactive-img-transform: scale(0.95) !important;

border-radius: 3.75em 3.75em 4.375em 4.375em !important;
border-radius: 3.75em 3.75em 50% 50% / 3.75em 3.75em 4.375em 4.375em !important;
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Would it work if it only has one value for horizontal and vertical radii if the bottom ones are larger than the img-option-container (just 3.75em 3.75em 4.5em 4.5em)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought using this strategy would mess up the prompt section for img quiz/polls with 2 options, but I tested it and it works because there's a small region where the chip is straight along the sides.

I just pushed the changes.

Copy link
Contributor

@mszylkowski mszylkowski left a comment

Choose a reason for hiding this comment

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

Let's 馃殺 it

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.

None yet

4 participants