Summary: | OPAC MARC details holdings table is not styled with thead | ||
---|---|---|---|
Product: | Koha | Reporter: | Aleisha Amohia <aleisha> |
Component: | OPAC | Assignee: | Aleisha Amohia <aleisha> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | katrin.fischer, nick |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | Sponsored | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Bug 18949: Adding thead tags to OPAC MARC details holdings table
[SIGNED-OFF] Bug 18949: Adding thead tags to OPAC MARC details holdings table Bug 18949: Adding thead tags to OPAC MARC details holdings table |
Description
Aleisha Amohia
2017-07-17 22:22:52 UTC
Created attachment 65087 [details] [review] Bug 18949: Adding thead tags to OPAC MARC details holdings table So that it can be styled alongside other tables in the OPAC. To test: 1) Log into staff side 2) Find OPACUserCSS syspref and add the following CSS: th, .table-bordered>thead>tr:first-child>th { color: red; } 3) Log into the OPAC 4) Go to your fines, your search history, your reading history etc - notice all of these table's headings are styled with red text 5) Go to the detail page of an item, then the MARC view. Notice this table's headings are now styled with red text 6) Apply patch and refresh page 7) Your holdings table should now have red headings. (You can delete the CSS from OPACUserCSS if you'd like.) Sponsored-by: Catalyst IT Created attachment 65391 [details] [review] [SIGNED-OFF] Bug 18949: Adding thead tags to OPAC MARC details holdings table So that it can be styled alongside other tables in the OPAC. To test: 1) Log into staff side 2) Find OPACUserCSS syspref and add the following CSS: th, .table-bordered>thead>tr:first-child>th { color: red; } 3) Log into the OPAC 4) Go to your fines, your search history, your reading history etc - notice all of these table's headings are styled with red text 5) Go to the detail page of an item, then the MARC view. Notice this table's headings are now styled with red text 6) Apply patch and refresh page 7) Your holdings table should now have red headings. (You can delete the CSS from OPACUserCSS if you'd like.) Sponsored-by: Catalyst IT Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 68457 [details] [review] Bug 18949: Adding thead tags to OPAC MARC details holdings table So that it can be styled alongside other tables in the OPAC. To test: 1) Log into staff side 2) Find OPACUserCSS syspref and add the following CSS: th, .table-bordered>thead>tr:first-child>th { color: red; } 3) Log into the OPAC 4) Go to your fines, your search history, your reading history etc - notice all of these table's headings are styled with red text 5) Go to the detail page of an item, then the MARC view. Notice this table's headings are now styled with red text 6) Apply patch and refresh page 7) Your holdings table should now have red headings. (You can delete the CSS from OPACUserCSS if you'd like.) Sponsored-by: Catalyst IT Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> I don't actually see a change following the test plan, but code change makes sense Pushed to master for 17.11, thanks to everybody involved! |