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
Account and email-prefs page update #19612
Conversation
PRbuilds results: Screenshots 💚 Exceptions 💚 A11y validation 💚 Microdata Validation Apache Benchmark Load Testing LightHouse Reporting --automated message |
@@ -37,6 +37,7 @@ $checkbox-size: $gs-gutter / 1.25; | |||
@include mq(desktop) { | |||
grid-template-columns: repeat(2, 1fr); | |||
grid-auto-rows: 1fr; | |||
width: gs-span(8); |
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.
this component should not know about with at this scope (that way it can be added to any pages of any width). It will however take the full width of the page by default!
Closing this as it has been superseded by #19726 |
What does this change?
What is the value of this and can you measure success?
Does this affect other platforms - Amp, Apps, etc?
No
Does this affect GLabs Paid Content Pages? Should it have support for Paid Content?
Screenshots
Email prefs page
Account page
Tested in CODE?