Replies: 4 comments 3 replies
-
I'll come back to the question of the when, but in terms of the how, there is a lot of opportunity here. It seems that the design goal can be split into two areas: The CardWe'd need to put together some kind of logic to decide whether a card invites you to 'share your story' or not, depending on the requirements, this could either be done up stream in facia-press in Frontend, or in DCR at the enhance / data transformation stage. My inclination would be to suggest that this visual change exists separately to the container styling changes (e.g a 'share your story' card not in the new container designs would still get the little 'share your story' badge) - but this is mostly for semantics, and there likely wouldn't be a technical limitation to either approach One thing I would note on the design, is that the yellow background covering the image is something we've stopped doing in DCR, with the stars (as an example) moving down below the title: You may want to consider this to ensure consistency in card design! EDIT: Just noticed the 'how long the callout is open for thing' - this increases the likelihood we'd want to make a change in facia-press to ensure we have this data! The ContainerFirst of all, I'd note that there is already a 'description' field supported in facia tool & fronts: This is great for us because it means the changes are almost entirely stylistic! We already have a method in DCR of allowing styles to be changed for specific containers, based on how they are tagged: https://github.com/guardian/dotcom-rendering/blob/main/dotcom-rendering/src/web/lib/decideContainerOverrides.ts
TL;DR - this all seems doable to me! The only problem could be the roll-out of DCR, especially as while we're doing early % tests, we'd likely be hesitant to risk the like-for-like-ness of the migration. |
Beta Was this translation helpful? Give feedback.
-
Hi Journalism 👋 Thanks for asking for our feedback, and thanks for this really well written RFC! Echoing @OllysCoding comments, I think these changes are very possible and I don't see any major issues here. Point by point: Container changes
DescriptionAs Olly mentions, there already exists a way to add a description to containers. I tested this in training and it just works Other style changesAdditional style variation (including the background image) could potentially be achieved using container tags (aka 'palettes') These palettes are traditionally thought of as ways to give existing containers a 'feel', like The background image could be added using the same palette property. The simplest would be to hardcode the svg in the source. Card changes
Varying the style of a card when the tone tag of Journalism have recent experience of adding a new design tag with the
As above, varying based on |
Beta Was this translation helpful? Give feedback.
-
Actions Based on the comments here I think we, Dotcom, have some actions:
@alinaboghiu @tkgnm are you happy for us to work on these or would you prefer to lead? |
Beta Was this translation helpful? Give feedback.
-
@tkgnm closing this as I think through all of yours and others good work we're done! |
Beta Was this translation helpful? Give feedback.
-
Motivation
In the journalism team, one of our OKR's for this quarter is to encourage our users to contribute to callouts, which are special articles that primarily exist to contain a form that readers can use to share their experiences with The Guardian.
Problem
Currently, there is not a very clear way of visually setting apart what is a "callout" article on fronts. Because of this, some of our users currently think callout articles are actual articles. This ambiguity could be having a negative impact on our users' experience. Additionally, we think that if we can make it clearer to our users that an article is a callout, we could drive user engagement in our callout campaigns.
Solution
We would like to change the way fronts renders callout containers/articles. See the below figma (WIP):
There are a few changes here
Fronts -> DCR migration
If possible, we'd like to only implement these changes on dotcom rendering to save us having to make changes in two codebases as we migrate to fronts rendering. This depends on the estimated rollout of fronts rendering.
What we'd like to know
Beta Was this translation helpful? Give feedback.
All reactions