-
Notifications
You must be signed in to change notification settings - Fork 554
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
SpecialReportAlt improvements / visual bug fixes #26018
Conversation
.fc-sublinks { | ||
.fc-sublink__title { | ||
&:before { | ||
border-top: 1px solid rgba(118, 118, 118, .3); |
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.
.fc-item__timestamp, | ||
.fc-trail__count--commentcount { | ||
background-color: transparent; | ||
} |
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.
…d has no SpecialReportAlt pillar
460ea67
to
9d5e720
Compare
.fc-item.fc-item--type-comment.fc-item--pillar-news { | ||
&.fc-item--type-comment { | ||
.fc-item__container.u-faux-block-link--hover { | ||
background-color: darken($special-report-alt-pastel, 5%); |
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.
Before | After |
---|---|
These two issues with the fc-item--type-comment
exist in all the palettes. We possibly haven't noticed them because the issues go away once a special report or special report alt tag is added to the article. For the purposes of SpecialReportAlt, I am fixing the avatar background for all the palettes and the hover state only for SpecialReportAltPalette.
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.
👍
Overdue on FRONTS-PROD, ADMIN-PROD (merged by @ioannakok 31 minutes and 56 seconds ago) What's gone wrong? |
Seen on FRONTS-PROD, ADMIN-PROD (merged by @ioannakok 38 minutes and 33 seconds ago)
|
What does this change?
Fixes a couple of visual bugs for SpecialReportAlt. See comments for screenshots.
Does this change need to be reproduced in dotcom-rendering ?
Screenshots
What is the value of this and can you measure success?
Checklist
Does this affect other platforms?
Does this affect GLabs Paid Content Pages? Should it have support for Paid Content?
Does this change break ad-free?
Does this change update the version of CAPI we're using?
Accessibility test checklist
Tested