-
Notifications
You must be signed in to change notification settings - Fork 10
Conversation
…publish date field to use as field to print the publishing date
…nged order of title and publish date
… and put styling into own component.
…header region of view and added publish date to teaser view mode
…ed news articles will go in here
a7482a4
to
aabe834
Compare
…d related articles view
@Heleen-cr is that going in for tomorrow release? |
trying to. What time do you need it the latest? |
Ideally tomorrow before standup, but we can wait until end of the day tomorrow |
<div class ='cr-article__image'> | ||
{{ field_article_image }} | ||
</div> | ||
<div class ='cr-article__video'> |
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.
@Heleen-cr I think we need to add conditions here if not it will show the markup when field is empty
<div class="related-articles__item"> {{ row }} </div> | ||
{% endfor %} | ||
</div> | ||
<div class="related-articles__footer"> |
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.
condition here to not show the markup now that we don't have footer
} | ||
} | ||
@include breakpoint($screen-md){ | ||
padding: $row-padding--md; |
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.
there is no selector for this rule
…actored at some point. Added some conditions to template file news article and related articles view. Commented out social links for now
…not in the design
Fixes https://jira.comicrelief.com/browse/PLAT-471
Changes proposed in this pull request