Update notification banner to be flexible in what it accepts about govuk-components HOT 5 CLOSED

I think you should be able to do this now by passing in your content with a block:

<%= govuk_notification_banner(blah) do %>
  <h2>Inner content goes here</h2>
<% end %>

from govuk-components.

Ah, nice. Is there an example of this usage in the docs?

from govuk-components.

Not exactly, I'll try to get it updated at some point - the whole docs could do with a re-jig.

from govuk-components.

oh yea i remember now sorry. I think i had this problem before and now going thru the same hoops due to my poor memory. to do the above we subclassed the component and override def render? to always return true. in our case we can't use the given heading as it doesn't fit our format. but in order for the component to render it must have a heading hence why we override this

from govuk-components.

Ah! To be honest I think I added the render? method before thinking about allowing arbitrary content via blocks. I'll just remove it, can totally see why it's a pain in the rear 😅

from govuk-components.