The patron card creator home and manage pages use an include, patroncards-toolbar.inc, which uses YUI to style toolbar buttons. It should use Bootstrap instead.
Created attachment 15625 [details] [review] Bug 9698 - Replace YUI buttons on patron card creator manage toolbar with Bootstrap This patch converts the toolbar include file used by the patron card creator home and manage pages 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"). To test, view the patron card creator home page and any manage page ("Manage layouts," "Manage profiles," etc). Buttons should look correct and work correctly.
Created attachment 15641 [details] [review] [SIGNED-OFF] Bug 9698 - Replace YUI buttons on patron card creator manage toolbar with Bootstrap This patch converts the toolbar include file used by the patron card creator home and manage pages 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"). To test, view the patron card creator home page and any manage page ("Manage layouts," "Manage profiles," etc). Buttons should look correct and work correctly. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Work as described. No errors. No "plus" sign for this buttons?
Created attachment 15874 [details] [review] [PASSED QA] Bug 9698 - Replace YUI buttons on patron card creator manage toolbar with Bootstrap This patch converts the toolbar include file used by the patron card creator home and manage pages 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"). To test, view the patron card creator home page and any manage page ("Manage layouts," "Manage profiles," etc). Buttons should look correct and work correctly. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Work as described. No errors. No "plus" sign for this buttons? Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> No plus signs? :) Works nicely and didn't have the plus before, so it's ok.
This patch has been pushed to master. Plus sign icons would be a nice follow-up.