Skip to content

Add warning text extension#78

Merged
bravegrape merged 1 commit into
masterfrom
add-warning-text-partial
Mar 20, 2019
Merged

Add warning text extension#78
bravegrape merged 1 commit into
masterfrom
add-warning-text-partial

Conversation

@vixus0
Copy link
Copy Markdown
Contributor

@vixus0 vixus0 commented Mar 19, 2019

Now you too can warn people as recommended by the GOV.UK Design System:
https://design-system.service.gov.uk/components/warning-text/

All you have to do is include: <%= warning_text('Look out!') %> in
your ERB content.

@bravegrape
Copy link
Copy Markdown

Now you too can warn people as recommended by the GOV.UK Design System:
https://design-system.service.gov.uk/components/warning-text/

All you have to do is include: `<%= warning_text('Look out!') %>` in
your ERB content.
@vixus0 vixus0 force-pushed the add-warning-text-partial branch from 2e7cfe9 to b650d31 Compare March 19, 2019 17:20
@bravegrape
Copy link
Copy Markdown

When this feature goes live, we should also add documentation to https://tdt-documentation.london.cloudapps.digital/ 👌

tlwr
tlwr previously approved these changes Mar 20, 2019
@tlwr tlwr dismissed their stale review March 20, 2019 10:09

It is not my place to approve this but I endorse it

Copy link
Copy Markdown
Contributor

@jakubmiarka jakubmiarka left a comment

Choose a reason for hiding this comment

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

Looks ok 👍 while #34 is being worked on

font-size: 16px;
font-size: 1rem;
line-height: 1.25;
color: #0b0c0c;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should probably be using colours from the palette and other stuff.

@bravegrape bravegrape merged commit 20dfebd into master Mar 20, 2019
@bravegrape bravegrape deleted the add-warning-text-partial branch March 20, 2019 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants