don't use primary for "a" css styling

Bug #1736967 reported by Rogan Hamby
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

There are a number of things that could be done to make the CSS more customization friendly in Evergreen but this is the one I run into most often and that is the "a" element in var/templates/opac/css/style.ccs.tt2 (~ line 37) uses css_colors.primary

This causes issues when the the primary colors for the header and footer are set to the same color as the center background and you then end up with the same color on the same background and this effects trickles into links throughout the opac.

For a partial fix I'm thinking of adding an entry to templates/opac/parts/css/colors.tt2 for a_links with a default value of "#ffffff" to match stock but then make it easily customizable.

Changed in evergreen:
assignee: nobody → Rogan Hamby (rogan-hamby)
Changed in evergreen:
assignee: Rogan Hamby (rogan-hamby) → nobody
Revision history for this message
Shula Link (slink-g) wrote :

Fix based on Roger's idea pushed here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/slink/lp1736967_primary_a_css_fix

Since the Angular 2 update seems to pull css from the bootstrap-css-only package, I'm not sure if there's a need for an implementation there.

tags: added: pullrequest
Shula Link (slink-g)
Changed in evergreen:
status: New → Confirmed
Changed in evergreen:
importance: Undecided → Low
Michele Morgan (mmorgan)
Changed in evergreen:
milestone: none → 3.next
Galen Charlton (gmc)
Changed in evergreen:
importance: Low → Wishlist
milestone: 3.next → 3.4-beta1
Changed in evergreen:
assignee: nobody → Rogan Hamby (rogan-hamby)
Revision history for this message
Rogan Hamby (rogan-hamby) wrote :

sign off pushed to user/rogan/lp1736967_signoff

tags: added: signedoff
Changed in evergreen:
assignee: Rogan Hamby (rogan-hamby) → nobody
Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master, along with a follow-up that adds release notes. Thanks, Sam and Rogan!

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Galen Charlton (gmc) → nobody
Galen Charlton (gmc)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.