Skip to content

avoid btn-warning

Bengfort requested to merge avoid-btn-warning into main

For a time we were thinking about using btn-warning in a few places. However, most related ideas fizzled out eventually (e.g. !2147 (closed)) and the only one that was actually merged was !2141 (merged). That button now feels a bit out of place.

Actually there is a second, older on from !2011 (merged). However, it is part of a warning alert, so the context is different there.

I propose to downgrade btn-warning to btn-secondary and not use it in new code. I think btn-secondary is better than btn-primary because it, like btn-warning, tells the user that this is not a button they should use by default.


Edited by Bengfort

Merge request reports