Attachment 163650 Details for Bug 36390 – Bug 36390: Two minor OPAC CSS fixes

[patch] Bug 36390: Two minor OPAC CSS fixes

Bug-36390-Two-minor-OPAC-CSS-fixes.patch (text/plain), 2.28 KB, created by Lucas Gass (lukeg) on 2024-03-21 19:45:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Lucas Gass (lukeg)

Created: 2024-03-21 19:45:44 UTC

Size: 2.28 KB

patch

obsolete

>From fb0af868d14bd0c85f5a3c91c819e3ca1edccb82 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 29 Feb 2024 16:36:34 +0000
>Subject: [PATCH] Bug 36390: Two minor OPAC CSS fixes
>
>This patch makes corrections to OPAC CSS to address two small issues.
>
>To test, apply the patch and rebuild the OPAC CSS.
>
>- Perform a catalog search which will return multiple results..
>- In the table of search results, check the icon and text for the
>  controls under each title: "Place hold", "Add tag", "Add to cart" etc.
>  - The colors should be consistent, with a slightly darker blue for the
>    icons.
>- Scroll down until the table header "sticks" to the top of the
>  viewport.
>- There should be no gap between the header row with the pagination
>  links and the row with the "Select all", "Clear all", etc. controls.
>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>---
> koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss b/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>index 809383f7c7..2a03a8fa42 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>+++ b/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>@@ -518,14 +518,16 @@ th {
> 
>         &:hover {
>             i {
>-                &.fa {
>+                &.fa,
>+                &.fa-solid {
>                     color: #338468;
>                 }
>             }
>         }
> 
>         i {
>-            &.fa {
>+            &.fa,
>+            &.fa-solid {
>                 color: #4466AE;
>             }
>         }
>@@ -535,7 +537,8 @@ th {
>                 color: #900;
> 
>                 i {
>-                    &.fa {
>+                    &.fa,
>+                    &.fa-solid {
>                         color: #C60000;
>                     }
>                 }
>@@ -549,7 +552,8 @@ th {
> 
>             &:hover {
>                 i {
>-                    &.fa {
>+                    &.fa,
>+                    &.fa-solid {
>                         color: #333;
>                         opacity: .65;
>                     }
>@@ -1167,7 +1171,6 @@ fieldset {
>     border-bottom: none;
>     display: flex;
>     margin: 0;
>-    margin-top: 3px;
>     padding-left: 10px;
>     padding-top: .5em;
>     flex-wrap: wrap;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 36390: 163649 | 163650 | 163651