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
Adding guidance link #1964
Adding guidance link #1964
Conversation
app/views/coronavirus_landing_page/components/shared/_country_section.html.erb
Outdated
Show resolved
Hide resolved
<% if guidance["intro"].present? %> | ||
<p class="govuk-body"> | ||
<%= render_govspeak(guidance["intro"]) %> | ||
<p> | ||
<% end %> |
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.
I think this should be moved outside of this if
as if there are no links this won't show
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.
Right, so take this outside of this <% if guidance.present? && guidance["links"].present? %>
check? So if the other links aren't present this paragraph will still render?
I'm wondering if the level of abstraction present in this partial could be reduced so there's editable content but it's more reflective of the rendered markup.
app/views/coronavirus_landing_page/components/shared/_country_section.html.erb
Outdated
Show resolved
Hide resolved
efa86d5
to
fa21222
Compare
@DilwoarH Ace - would it be possible to merge when ready? I'm not authorised |
f63c7f7
to
eed320b
Compare
Adding new guidance link to landing page Co-authored-by: Dilwoar Hussain <dilwoar.hussain@digital.cabinet-office.gov.uk>
eed320b
to
5345d9d
Compare
What?
Why
Trello Ticket
Visual Before
Visual After
Anything else?
Content was updated here (& here) as a precursor.
Actual link directs:
/search/all?level_one_taxon=5b7b9532-a775-4bd2-a3aa-6ce380184b6c&content_purpose_supergroup%5B%5D=guidance_and_regulation&order=most-viewed