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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix clock in SpecialReportAlt card #25992

Closed
wants to merge 9 commits into from

Conversation

ioannakok
Copy link
Contributor

@ioannakok ioannakok commented Mar 20, 2023

What does this change?

Index fronts show a clock and time of publication. This PR fixes those for SpecialReportAlt in order to match the designs.

Does this change need to be reproduced in dotcom-rendering ?

Screenshots

Before After
image image

Other pillars stay the same

Before After
image image
image image

What is the value of this and can you measure success?

Checklist

Does this affect other platforms?

  • AMP
  • Apps
  • Other (please specify)

Does this affect GLabs Paid Content Pages? Should it have support for Paid Content?

  • No
  • Yes (please give details)

Does this change break ad-free?

  • No
  • It did, but tests caught it and I fixed it
  • It did, but there was no test coverage so I added that then fixed it

Does this change update the version of CAPI we're using?

Accessibility test checklist

Tested

  • Locally
  • On CODE (optional)

@ioannakok ioannakok force-pushed the ioanna/fix-cardstyle-for-special-report-alt branch from c0d7dd4 to dfcb697 Compare March 21, 2023 09:09
Base automatically changed from ioanna/fix-cardstyle-for-special-report-alt to main March 21, 2023 10:14
</span>
</time>
} else {
<time class="fc-item__timestamp@if(timeStampDisplay.javaScriptUpdate) { js-item__timestamp}"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

How to do else here?

@ioannakok
Copy link
Contributor Author

ioannakok commented Mar 27, 2023

Fixed with this: #25984

@ioannakok ioannakok closed this Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant