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

Guardian news provider of the year #27065

Merged
merged 2 commits into from
Apr 19, 2024
Merged

Guardian news provider of the year #27065

merged 2 commits into from
Apr 19, 2024

Conversation

ioannakok
Copy link
Contributor

@ioannakok ioannakok commented Apr 18, 2024

What does this change?

Adds Guardian News Provider logo

Change is DCR:

Screenshots

Wide

image

Tablet

image

image

Checklist

  • Tested locally, and on CODE if necessary

@ioannakok ioannakok requested a review from HarryFischer April 18, 2024 17:31
@domlander domlander marked this pull request as ready for review April 19, 2024 08:44
@domlander domlander requested a review from a team as a code owner April 19, 2024 08:44
@domlander domlander merged commit 0b5be5b into main Apr 19, 2024
2 checks passed
@domlander domlander deleted the guardian-news-provider branch April 19, 2024 08:49
Copy link
Contributor

Choose a reason for hiding this comment

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

It's a bit of a shame that we make these single-use graphics when we have already loaded the fonts and have an agnostic base layer 🤷

@prout-bot
Copy link
Collaborator

Seen on FRONTS-PROD, ADMIN-PROD (created by @ioannakok and merged by @domlander 35 minutes and 19 seconds ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants