Created attachment 49661 [details] [review] Bug 16162: Making dictionary 'Delete' a button To test: 1) Go to Reports -> View dictionary 2) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button. 3) Confirm that the button does not wrap on narrower browsers. 4) Confirm that column heading is now 'Action'. 5) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.) Sponsored-by: Catalyst IT
QA tools complain: FAIL koha-tmpl/intranet-tmpl/prog/en/modules/reports/dictionary.tt FAIL forbidden patterns forbidden pattern: tab char (line 82) forbidden pattern: tab char (line 71) forbidden pattern: tab char (line 79)
Created attachment 49713 [details] [review] Bug 16162: Making dictionary 'Delete' a button EDIT: This patch should have gotten rid of those tabs To test: 1) Go to Reports -> View dictionary 2) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button. 3) Confirm that the button does not wrap on narrower browsers. 4) Confirm that column heading is now 'Action'. 5) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.) Sponsored-by: Catalyst IT
Created attachment 49715 [details] [review] Bug 16162: Making dictionary 'Delete' a button EDIT: This patch should have gotten rid of those tabs To test: 1) Go to Reports -> View dictionary 2) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button. 3) Confirm that the button does not wrap on narrower browsers. 4) Confirm that column heading is now 'Action'. 5) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.) Sponsored-by: Catalyst IT Signed-off-by: Marc Véron <veron@veron.ch>
Aleisha, please submit a followup on top of bug 16161
Created attachment 49748 [details] [review] Bug 16162: Making dictionary 'Delete' a button EDIT: This patch should have gotten rid of those tabs EDIT2: Will now apply with patch on Bug 16161 To test: 1) Apply Bug 16161 first 2) Go to Reports -> View dictionary 3) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button. 4) Confirm that the button does not wrap on narrower browsers. 5) Confirm that column heading is now 'Action'. 6) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.) Sponsored-by: Catalyst IT Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 49816 [details] [review] Bug 16162: Making dictionary 'Delete' a button EDIT: This patch should have gotten rid of those tabs EDIT2: Will now apply with patch on Bug 16161 To test: 1) Apply Bug 16161 first 2) Go to Reports -> View dictionary 3) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button. 4) Confirm that the button does not wrap on narrower browsers. 5) Confirm that column heading is now 'Action'. 6) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.) Sponsored-by: Catalyst IT Tested on top of Bug 16161 Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 49819 [details] [review] [PASSED QA] Bug 16162: Making dictionary 'Delete' a button EDIT: This patch should have gotten rid of those tabs EDIT2: Will now apply with patch on Bug 16161 To test: 1) Apply Bug 16161 first 2) Go to Reports -> View dictionary 3) Confirm that the 'Delete definition' button now just says 'Delete' and is styled as a font awesome button. 4) Confirm that the button does not wrap on narrower browsers. 5) Confirm that column heading is now 'Action'. 6) Confirm that button deletes as expected. (If you're wondering about the missing confirm message upon delete, see Bug 16161.) Sponsored-by: Catalyst IT Tested on top of Bug 16161 Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to Master - Should be in the May 2016 release. Thanks!