Bug 32148: Buttons must inherit Bootstrap size classes
authorOwen Leonard <oleonard@myacpl.org>
Wed, 9 Nov 2022 14:09:17 +0000 (14:09 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Thu, 10 Nov 2022 12:28:19 +0000 (09:28 -0300)
This patch removes global font and padding settings which were
overriding the Bootstrap classes we use to change the size of Bootstrap
buttons. The font and padding settings are moved to the CSS which covers
button which are not styled as Bootstrap buttons.

To test, apply the patch and view pages where buttons are styled with
different Bootstrap size classes:

- Circulation -> Check out:
  - The "Check out" button, the "Show checkouts" button, Renew buttons
    in the footer.
- Patrons -> Patron details -> Edit buttons in each section.
- Patrons home -> Buttons in sidebar filter
- Catalog -> Bibliographic details -> Item details

Signed-off-by: David Nind <david@davidnind.com>
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/_mixins.scss
koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss

index 4b8b773..d5924ac 100644 (file)
@@ -13,11 +13,9 @@ $nav-menu-bullet: url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/s
     border-radius: 4px;
     cursor: pointer;
     display: inline-block;
-    font-size: inherit;
     font-weight: normal;
     line-height: 1.42857143;
     margin-bottom: 0;
-    padding: .5em 1em;
     text-align: center;
     touch-action: manipulation;
     user-select: none;
@@ -44,11 +42,9 @@ $nav-menu-bullet: url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/s
     border-radius: 4px;
     cursor: pointer;
     display: inline-block;
-    font-size: inherit;
     font-weight: normal;
     line-height: 1.42857143;
     margin-bottom: 0;
-    padding: .5em 1em;
     text-align: center;
     touch-action: manipulation;
     user-select: none;
index f41f3d7..f0b7640 100644 (file)
@@ -570,11 +570,13 @@ textarea {
 input {
     &[type="submit"] {
         @include primary-button;
+        padding: .5em 1em;
     }
 
     &[type="reset"],
     &[type="button"] {
         @include default-button;
+        padding: .5em 1em;
 
         &:disabled {
             @include disabled-button;
@@ -590,6 +592,7 @@ input {
 
     &.submit {
         @include primary-button;
+        padding: .5em 1em;
 
         &:disabled {
             @include disabled-button;