[Bug 9636] New: Replace YUI buttons on lables pages with Bootstrap
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Bug ID: 9636 Summary: Replace YUI buttons on lables pages with Bootstrap Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All URL: /cgi-bin/koha/labels/label-home.pl OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org The toolbar include file used on labels pages uses YUI to generate buttons and menus. It should use Bootstrap instead. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Replace YUI buttons on |Replace YUI buttons on |lables pages with Bootstrap |labels pages with Bootstrap -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 15398 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15398&action=edit Bug 9636 - Replace YUI buttons on labels pages with Bootstrap This patch converts the toolbar include file used by the label manager to Bootstrap, replacing YUI button and menu code with Bootstrap markup. This patch converts the previous menu buttons to plain buttons, removing the redundant links to the manage pages, which are in the left-hand sidebar. This keeps toolbar buttons within the general category of "action" (as opposed to "view"). This patch also removes an unused include file, labels-new-toolbar.inc To test, view the labels home page and any of the label manage pages (layouts, templates, profiles, batches). Buttons should look correct and work correctly. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15398|0 |1 is obsolete| | --- Comment #2 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 15406 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15406&action=edit [SIGNED-OFF] Bug 9636 - Replace YUI buttons on labels pages with Bootstrap This patch converts the toolbar include file used by the label manager to Bootstrap, replacing YUI button and menu code with Bootstrap markup. This patch converts the previous menu buttons to plain buttons, removing the redundant links to the manage pages, which are in the left-hand sidebar. This keeps toolbar buttons within the general category of "action" (as opposed to "view"). This patch also removes an unused include file, labels-new-toolbar.inc To test, view the labels home page and any of the label manage pages (layouts, templates, profiles, batches). Buttons should look correct and work correctly. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Work as described. No errors. Nicole is not going to be happy, images in Label creator section of manual needs to change :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |bgkriegel@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15406|0 |1 is obsolete| | --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 15456 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15456&action=edit [PASSED QA] Bug 9636 - Replace YUI buttons on labels pages with Bootstrap This patch converts the toolbar include file used by the label manager to Bootstrap, replacing YUI button and menu code with Bootstrap markup. This patch converts the previous menu buttons to plain buttons, removing the redundant links to the manage pages, which are in the left-hand sidebar. This keeps toolbar buttons within the general category of "action" (as opposed to "view"). This patch also removes an unused include file, labels-new-toolbar.inc To test, view the labels home page and any of the label manage pages (layouts, templates, profiles, batches). Buttons should look correct and work correctly. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Work as described. No errors. Nicole is not going to be happy, images in Label creator section of manual needs to change :) Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Works nicely, I was wondering if we should have the plus icon on those buttons like we have on the other 'New' buttons. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de QA Contact| |katrin.fischer@bsz-bw.de --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I like the changes to the menu structure. My only question is, why the buttons don't have the plus icons like we have it on the other 'new' buttons. No reason not to pass QA on this improvement tho :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9636 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #5 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org