-
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
Fix various SpecialReportAlt bugs #26003
Conversation
@@ -415,7 +415,7 @@ $pillars: ( | |||
background-color: $special-report-alt-faded; | |||
|
|||
.fc-item__container.u-faux-block-link--hover { | |||
background-color: darken($special-report-alt-faded, 2%); | |||
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.
.fc-item--pillar-special-report-alt { | ||
.fc-item__meta { | ||
.inline-icon { | ||
fill: $special-report-alt-dark; |
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: 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.
} | ||
} | ||
|
||
.fc-item--pillar-special-report-alt { | ||
.fc-item__container > .fc-item__meta { | ||
@include multiline(3, rgba(60, 60, 60, .3)); | ||
@include multiline(3, rgba(112, 112, 112, .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.
Fixes the grey lines for opinion cards to be the same grey as all the lines / borders in the palette
@@ -11,3 +11,4 @@ | |||
@include overide-interview-headline-colours(arts, $culture-dark, $culture-pastel); | |||
@include overide-interview-headline-colours(lifestyle, $lifestyle-dark, $news-pastel); | |||
@include overide-interview-headline-colours(special-report, #ffffff, #ffffff); | |||
@include overide-interview-headline-colours(special-report-alt, $special-report-alt-dark, $special-report-alt-dark); |
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.
|
||
@if $pillar == special-report-alt { | ||
background-color: darken($color1, 2%); | ||
background-color: darken($special-report-alt-pastel, 5%); | ||
} @else { | ||
background-color: darken($color1, 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.
Duplicate code
|
||
@if $pillar == special-report-alt { | ||
background-color: darken($color1, 2%); | ||
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.
@@ -121,7 +121,7 @@ | |||
|
|||
&:hover, | |||
.u-faux-block-link--hover { | |||
background-color: darken($special-report-alt-faded, 2%); | |||
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.
Co-authored-by: Olly Namey <olly.namey@guardian.co.uk>
Co-authored-by: Olly Namey <olly.namey@guardian.co.uk>
Co-authored-by: Olly Namey <olly.namey@guardian.co.uk>
4bc000e
to
b47b675
Compare
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 looks good to me!
Seen on FRONTS-PROD, ADMIN-PROD (merged by @ioannakok 20 minutes and 43 seconds ago)
|
What does this change?
Fixes various visual bugs in SpecialReportAlt fronts cards:
Does this change need to be reproduced in dotcom-rendering ?
Screenshots
This is a more general screenshot but more detailed can be found in PR comments.
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