[BUG] Warning button isn't red enough

[catherinehunt]

The colour of the warning button is very tasteful and on-brand, but doesn’t really say ‘warning’ to me. At least on my monitor, it displays as quite brown. Ditto for the "danger" notice class.

Expected behaviour

Red-coloured button

Current behaviour

Red-brown button (russet??)

Steps to reproduce

<a class="button warning">Warning</a>

Affected browsers

• [X] Chrome
• [ ] Firefox
• [ ] Edge
• [ ] IE 11
• [ ] Safari 8+
• [ ] iOS 8.4+
• [ ] Android 4.4+

[jaredhenderson]

Hi @catherinehunt, you're spot on about the colour - it's actually the Orange Dark from our digital colour palette as we don't have any reds! Was before my time so I can't explain the reasoning for that.

Anyway, I completely agree and I think for the sake of clarity that we can make an exception in this case. Top button pictured below uses an updated red.

@makenosound this is probably the third time I've asked you to change this one, sorry

• [ ] Adjust warning button class to use red #DC291E
• [ ] Hover state colour can be #C11F1B

[tomstringer]

This one also needs a bit more attention.

Red works ok on buttons, though the current hover state probably needs review (currently #984140).

Notices have both warning and danger classes: https://web.unimelb.edu.au/components/notices/

Red background on danger looks a bit hectic.

[jaredhenderson]

@tomstringer cool, let's address the button itself here and I'll re-jig the notice ticket to resolve the other issues. I've amended my comment above with an additional value for the hover background colour.