Summary: | Replace YUI buttons and menus on serials pages with Bootstrap | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | bgkriegel |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
URL: | /cgi-bin/koha/serials/serials-home.pl | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 9627 - Replace YUI buttons and menus on serials pages with Bootstrap
[SIGNED-OFF] Bug 9627 - Replace YUI buttons and menus on serials pages with Bootstrap [PASSED QA] Bug 9627 - Replace YUI buttons and menus on serials pages with Bootstrap |
Description
Owen Leonard
2013-02-15 16:54:24 UTC
Created attachment 15380 [details] [review] Bug 9627 - Replace YUI buttons and menus on serials pages with Bootstrap This patch converts the toolbar include file used by Serials pages to Bootstrap, replacing YUI button and menu code with Bootstrap markup. To test, view any page in serials/ which uses serials-toolbar.inc (subscription-detail.pl, serials-home.pl, etc). Buttons and menus should look correct and work correctly, including: - New subscription - Edit subscription - Edit as new (duplicate) - Delete subscription - Renew - Receive - Close Created attachment 15389 [details] [review] [SIGNED-OFF] Bug 9627 - Replace YUI buttons and menus on serials pages with Bootstrap This patch converts the toolbar include file used by Serials pages to Bootstrap, replacing YUI button and menu code with Bootstrap markup. To test, view any page in serials/ which uses serials-toolbar.inc (subscription-detail.pl, serials-home.pl, etc). Buttons and menus should look correct and work correctly, including: - New subscription - Edit subscription - Edit as new (duplicate) - Delete subscription - Renew - Receive - Close Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Work as described. No errors. For me it would be better to use icon-remove, because icon-remove-circle shows a little cross, think that a bigger cross is more clear. Created attachment 15448 [details] [review] [PASSED QA] Bug 9627 - Replace YUI buttons and menus on serials pages with Bootstrap This patch converts the toolbar include file used by Serials pages to Bootstrap, replacing YUI button and menu code with Bootstrap markup. To test, view any page in serials/ which uses serials-toolbar.inc (subscription-detail.pl, serials-home.pl, etc). Buttons and menus should look correct and work correctly, including: - New subscription - Edit subscription - Edit as new (duplicate) - Delete subscription - Renew - Receive - Close Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Work as described. No errors. For me it would be better to use icon-remove, because icon-remove-circle shows a little cross, think that a bigger cross is more clear. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Works nicely, no problems found. I think there is a point in differentiating between closing and deleting, so the icon-remove-circle is maybe not ideal, but the icon-remove would falsely indicate a delete. Maybe we can find another icon that expresses the functionality a bit better later on. This patch has been pushed to master. |