| Summary: | OPAC Messaging Settings 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: | fridolin.somers, jesse |
| Version: | unspecified | Keywords: | Academy |
| 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: | ||
| Circulation function: | |||
| Attachments: |
Bug 18860: Adding thead tags to OPAC messaging settings table
Bug 18860: Adding thead tags to OPAC messaging settings table Bug 18860: Adding thead tags to OPAC messaging settings table |
||
|
Description
Aleisha Amohia
2017-06-27 01:57:23 UTC
Created attachment 64667 [details] [review] Bug 18860: Adding thead tags to OPAC messaging settings 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: thead { 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 your messaging. Notice this table's headings are not styled with red text 6) Apply patch and refresh page 7) Your messaging table should now have red headings. (You can delete the CSS from OPACUserCSS if you'd like.) Sponsored-by: Catalyst IT Created attachment 64852 [details] [review] Bug 18860: Adding thead tags to OPAC messaging settings 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: thead { 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 your messaging. Notice this table's headings are not styled with red text 6) Apply patch and refresh page 7) Your messaging table should now have red headings. (You can delete the CSS from OPACUserCSS if you'd like.) Sponsored-by: Catalyst IT Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com> This patch does exactly what the tests describe. Created attachment 66654 [details] [review] Bug 18860: Adding thead tags to OPAC messaging settings 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: thead { 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 your messaging. Notice this table's headings are not styled with red text 6) Apply patch and refresh page 7) Your messaging table should now have red headings. (You can delete the CSS from OPACUserCSS if you'd like.) Sponsored-by: Catalyst IT Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Pushed to master for 17.11, thanks to everybody involved! Enhancement not pushed to 17.05.x |