[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Owen Leonard changed: What|Removed |Added Patch complexity|--- |Small patch Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 --- Comment #1 from Owen Leonard --- Created attachment 81320 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81320&action=edit Bug 21693: Clean up checkout notes template This patch modifies the checkout notes template to help compliance with coding guidelines and interface patterns: - Convert to Bootstrap grid - Improve DataTables configuration - Put buttons into a toolbar and move above selection links Also corrected: Minor markup error in circ-nav.inc To test, apply the patch and enable the AllowCheckoutNotes system preference. Add a few checkout notes via the OPAC. - Go to Circulation -> Checkout notes. - Confirm that the page looks correct at various browser widths. - Confirm that the first and last columns of the table of notes are not sortable. - The table should be sorted by default by title. - Title sorting should ignore articles "a," "an," and "the." - Test with the CircSidebar preference both on and off. - With CircSidebar turn on, the checkout notes menu item in the left hand sidebar should show a count of checkout notes. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Owen Leonard changed: What|Removed |Added Attachment #81320|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |ASSIGNED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Owen Leonard changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 --- Comment #2 from Owen Leonard --- Created attachment 81321 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81321&action=edit Bug 21693: Clean up checkout notes template This patch modifies the checkout notes template to help compliance with coding guidelines and interface patterns: - Convert to Bootstrap grid - Improve DataTables configuration - Put buttons into a toolbar and move above selection links Also corrected: Minor markup error in circ-nav.inc To test, apply the patch and enable the AllowCheckoutNotes system preference. Add a few checkout notes via the OPAC. - Go to Circulation -> Checkout notes. - Confirm that the page looks correct at various browser widths. - Confirm that the first and last columns of the table of notes are not sortable. - The table should be sorted by default by title. - Title sorting should ignore articles "a," "an," and "the." - Test with the CircSidebar preference both on and off. - With CircSidebar turn on, the checkout notes menu item in the left hand sidebar should show a count of checkout notes. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Pierre-Marc Thibault changed: What|Removed |Added Attachment #81321|0 |1 is obsolete|| --- Comment #3 from Pierre-Marc Thibault --- Created attachment 81339 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81339&action=edit Bug 21693: Clean up checkout notes template This patch modifies the checkout notes template to help compliance with coding guidelines and interface patterns: - Convert to Bootstrap grid - Improve DataTables configuration - Put buttons into a toolbar and move above selection links Also corrected: Minor markup error in circ-nav.inc To test, apply the patch and enable the AllowCheckoutNotes system preference. Add a few checkout notes via the OPAC. - Go to Circulation -> Checkout notes. - Confirm that the page looks correct at various browser widths. - Confirm that the first and last columns of the table of notes are not sortable. - The table should be sorted by default by title. - Title sorting should ignore articles "a," "an," and "the." - Test with the CircSidebar preference both on and off. - With CircSidebar turn on, the checkout notes menu item in the left hand sidebar should show a count of checkout notes. Signed-off-by: Pierre-Marc Thibault -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Pierre-Marc Thibault changed: What|Removed |Added CC||pierre-marc.thibault@inlibr ||o.com Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Katrin Fischer changed: What|Removed |Added Attachment #81339|0 |1 is obsolete|| --- Comment #4 from Katrin Fischer --- Created attachment 81363 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81363&action=edit Bug 21693: Clean up checkout notes template This patch modifies the checkout notes template to help compliance with coding guidelines and interface patterns: - Convert to Bootstrap grid - Improve DataTables configuration - Put buttons into a toolbar and move above selection links Also corrected: Minor markup error in circ-nav.inc To test, apply the patch and enable the AllowCheckoutNotes system preference. Add a few checkout notes via the OPAC. - Go to Circulation -> Checkout notes. - Confirm that the page looks correct at various browser widths. - Confirm that the first and last columns of the table of notes are not sortable. - The table should be sorted by default by title. - Title sorting should ignore articles "a," "an," and "the." - Test with the CircSidebar preference both on and off. - With CircSidebar turn on, the checkout notes menu item in the left hand sidebar should show a count of checkout notes. Signed-off-by: Pierre-Marc Thibault Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 --- Comment #5 from Katrin Fischer --- Created attachment 81364 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81364&action=edit Bug 21693: Make page title consistent with other pages For some reason we use 'Home' in the bread crumbs, but 'Koha' as starting point in the page title. Changed to make consistent with other pages in Circulation. Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Passed QA |Failed QA --- Comment #6 from Nick Clemens --- Before this patch the 'Mark seen' and 'Mark not seen' buttons work. I can't make them work with this -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Owen Leonard changed: What|Removed |Added Attachment #81363|0 |1 is obsolete|| Attachment #81364|0 |1 is obsolete|| --- Comment #7 from Owen Leonard --- Created attachment 82226 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82226&action=edit Bug 21693: Clean up checkout notes template This patch modifies the checkout notes template to help compliance with coding guidelines and interface patterns: - Convert to Bootstrap grid - Improve DataTables configuration - Put buttons into a toolbar and move above selection links Also corrected: Minor markup error in circ-nav.inc To test, apply the patch and enable the AllowCheckoutNotes system preference. Add a few checkout notes via the OPAC. - Go to Circulation -> Checkout notes. - Confirm that the page looks correct at various browser widths. - Confirm that the first and last columns of the table of notes are not sortable. - The table should be sorted by default by title. - Title sorting should ignore articles "a," "an," and "the." - Test with the CircSidebar preference both on and off. - With CircSidebar turn on, the checkout notes menu item in the left hand sidebar should show a count of checkout notes. Signed-off-by: Pierre-Marc Thibault Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 --- Comment #8 from Owen Leonard --- Created attachment 82227 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82227&action=edit Bug 21693: Make page title consistent with other pages For some reason we use 'Home' in the bread crumbs, but 'Koha' as starting point in the page title. Changed to make consistent with other pages in Circulation. Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Owen Leonard changed: What|Removed |Added Status|Failed QA |Passed QA --- Comment #9 from Owen Leonard --- (In reply to Nick Clemens from comment #6) > Before this patch the 'Mark seen' and 'Mark not seen' buttons work. I can't > make them work with this This was an error in the placement of the tag. That's the only change to the signed-off patch. I hope it's okay to return this to 'Passed QA.' -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 --- Comment #10 from Nick Clemens --- (In reply to Owen Leonard from comment #9) > (In reply to Nick Clemens from comment #6) > > Before this patch the 'Mark seen' and 'Mark not seen' buttons work. I can't > > make them work with this > > This was an error in the placement of the tag. That's the only change > to the signed-off patch. I hope it's okay to return this to 'Passed QA.' Yup,that's okay, would be better as a follow-up patch so that the change can be easily seen and verified, but it works in testing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Nick Clemens changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #11 from Nick Clemens --- Awesome work all! Pushed to master for 19.05 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21693] Clean up checkout notes template
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21693 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Status|Pushed to Master|RESOLVED Resolution|--- |FIXED --- Comment #12 from Martin Renvoize --- Enhancement, will not be backported to 18.11.x series. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/