Created attachment 58744 [details] [review] Bug 17874: Bug 16239 followup - polishing things This patch is made on top of bug 16239 to polish things. Makes these changes: - Buttons on add circulation message modal - Dropdowns has top and bottom margin 4px instead of 3px - btn-link class (for example in toolbar on advanced search pages) - Delete circ message is btn-link - Add new circ message is btn-link Test plan: - Confirm that all changes are as described above and that they make sense
First followup for bug 16239, I am sure there are many other things to do, but it would be useful to have any feedback on this and on the other possible changes
Created attachment 58807 [details] [review] Bug 17874: Bug 16239 followup - polishing things This patch is made on top of bug 16239 to polish things. Makes these changes: - Buttons on add circulation message modal - Dropdowns has top and bottom margin 4px instead of 3px - btn-link class (for example in toolbar on advanced search pages) - Delete circ message is btn-link - Add new circ message is btn-link Test plan: - Confirm that all changes are as described above and that they make sense Signed-off-by: Claire Gravely <claire_gravely@hotmail.com>
Created attachment 58870 [details] [review] Bug 17874: Bug 16239 followup - polishing things This patch is made on top of bug 16239 to polish things. Makes these changes: - Buttons on add circulation message modal - Dropdowns has top and bottom margin 4px instead of 3px - btn-link class (for example in toolbar on advanced search pages) - Delete circ message is btn-link - Add new circ message is btn-link Test plan: - Confirm that all changes are as described above and that they make sense Signed-off-by: Claire Gravely <claire_gravely@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Pushed to master for 17.05, thanks Josef!
Dependent on 16239 which is not in 16.11.x.