Summary: | Add class to items.itemnotes_nonpublic | ||
---|---|---|---|
Product: | Koha | Reporter: | Jessie Zairo <jzairo> |
Component: | Staff interface | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, jpalmer, jschmidt, m.de.rooy, nick |
Version: | unspecified | Keywords: | Academy |
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 19806 - Add classes to itemnotes in issues table
Bug 19806: Add classes to itemnotes in issues table Bug 19806: Add classes to itemnotes in issues table |
Description
Jessie Zairo
2017-12-13 19:15:13 UTC
Created attachment 69801 [details] [review] Bug 19806 - Add classes to itemnotes in issues table To test: 1 - Checkout an item with both public and non-public notes 2 - Note the notes display and both have circ-hlt class 3 - Apply patch 4 - Ensure display has not changed 5 - Note notes now have classes 'item-note-public' and 'item-note-nonpublic' 6 - Sign off Created attachment 69803 [details] [review] Bug 19806: Add classes to itemnotes in issues table To test: 1 - Checkout an item with both public and non-public notes 2 - Note the notes display and both have circ-hlt class 3 - Apply patch 4 - Ensure display has not changed 5 - Note notes now have classes 'item-note-public' and 'item-note-nonpublic' 6 - Sign off Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 69818 [details] [review] Bug 19806: Add classes to itemnotes in issues table To test: 1 - Checkout an item with both public and non-public notes 2 - Note the notes display and both have circ-hlt class 3 - Apply patch 4 - Ensure display has not changed 5 - Note notes now have classes 'item-note-public' and 'item-note-nonpublic' 6 - Sign off Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> The only thing I could say is that item-note- does not fully match the field name itemnotes :) Which is no problem to me! (In reply to Marcel de Rooy from comment #4) > The only thing I could say is that item-note- does not fully match the field > name itemnotes :) Which is no problem to me! Ah, yes, I copied the classes I found on returns.tt, figured the same css could be used consistently then. Pushed to master for 18.05, thanks to everybody involved! Awesome work all! Pushed to stable for 17.11.01 Pushed to 17.05.x for v17.05.10 |