Bug 32145: Fix style of buttons with class .btn-danger
authorOwen Leonard <oleonard@myacpl.org>
Thu, 10 Nov 2022 11:49:33 +0000 (11:49 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Thu, 10 Nov 2022 17:47:42 +0000 (14:47 -0300)
commitd6f1ca62555408819541c888f4fb19c3ef93e36d
treecf23f6f11d859d87144591dad870b452e834662b
parentd2cbcc205301b3b0d8c43003b891a7535d02a2cc
Bug 32145: Fix style of buttons with class .btn-danger

This patch adds style to the staff client's global SCSS which adds
correct styling to buttons classed with .btn-danger.

This isn't the correct way to fix this, but it's the fast way.
Rebuilding Bootstrap with custom colors is the correct way, but
Bootstrap 3.x doesn't use SCSS. I consider this is a FIXME for an
upgrade of Bootstrap in the staff interface.

To test, apply the patch and rebuild the staff interface CSS. Test
instances where the .btn-danger class is used. Two examples:

Holds:

- Locate a bibliographic records with holds and view the holds.
- Click the "trash" icon to delete one of the holds.
- In the confirmation modal, check that the "Confirm cancellation"
  button looks correct in its default, hover, and active ("clicked")
  state.

Cataloging:

- Open a new blank record in the basic MARC editor.
- Click "Save" without filling any mandatory fields.
- You should see a warning about the empty fields.
- Scroll down until the warning is offscreen. An "Errors" button should
  appear in the toolbar.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss