-
Notifications
You must be signed in to change notification settings - Fork 29
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Less article level css #1637
Less article level css #1637
Conversation
PRbuilds results: 💚 AMP validation LightHouse Reporting --automated message |
Size Change: -23 B (0%) Total Size: 514 kB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -43,42 +43,6 @@ const bodyStyle = (display: Display) => css` | |||
width: 100%; | |||
height: auto; | |||
} | |||
|
|||
figure { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this completely removed because we don't have a figure
used without styling being applied directly to the figure element?
What does this change?
Deletes the 'global'
ul
andli
css set at the article body level and drops it down to inside theTextBlockComponent
Why?
Because in photo essays the
ul
andli
tags are used to denote captions and we want to style them for that but this global pattern creates a css war. Rather than get in a fight I changed the scope of the original overrides to only apply to text blocks.