Summary: | Add distinct classes for clear history (.clearsh) links on OPAC | ||
---|---|---|---|
Product: | Koha | Reporter: | Lucas Gass (lukeg) <lucas> |
Component: | OPAC | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | trivial | ||
Priority: | P5 - low | ||
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: | Bug 25735: Add seperate classes for each clearsh button |
Description
Lucas Gass (lukeg)
2020-06-12 19:58:53 UTC
Created attachment 105850 [details] [review] Bug 25735: Add seperate classes for each clearsh button TEST PLAN: 1. Apply patch 2. Add these two lines to the OPACUserCSS: .clearsh-one {color:green !important;} .clearsh-two {color:red !important;} 3. Refresh OPAC and be logged out, the "Clear" button in the top right should now be green. 4. Refresh OPAC and be logged in, the "Clear" button inside the dropdown on the top right should be red. I hadn't notice this patch before because it was still marked "NEW." I think things may have changed a little since this was originally submitted. There is a fairly simple way to differentiate the two links now: /* Color for logged-out users */ .clearsh {color:green !important;} /* Color for logged-in users */ #loggedinuser-menu .clearsh { color:blue !important} I prefer this to adding the -one -two classes because they're not semantically meaningful. |