Bug #18937

Bst modal confirm button shouldn't be red unless action is destructive

Added by Walden Raines about 1 month ago. Updated about 1 month ago.

Status:Closed
Priority:Normal
Assigned To:Walden Raines
Category:-
Target version:Katello - Team Tom - iteration 11
Difficulty:trivial Pull request:https://github.com/Katello/bastion/pull/177
Bugzilla link:
Story points-
Velocity based estimate-

Description

We are using red modal buttons in multiple places (bookmark this search, CH errata apply, etc.) where the action is not destructive and shouldn't use a red button.

Associated revisions

Revision 1f502577
Added by Walden Raines about 1 month ago

Fixes #18937: allow modal confirm button to be specified.

Instead of assuming all modals have a .btn-danger we should allow the
confirm button to be specified.

http://projects.theforeman.org/issues/18937

History

#1 Updated by The Foreman Bot about 1 month ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/bastion/pull/177 added

#2 Updated by Walden Raines about 1 month ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF