Created attachment 29687 [details] link color hard to read When on the advanced search the links next to the search box need a class or ID so that they can be styled. For example, the link color in the attached image would be better white, but there is no ID to grab on to like this is on the moresearches box at the top.
Created attachment 30065 [details] [review] proposed patch I have added bootstrap classes to the links, and made them 'btn-default' which means they will appear grey (and less prominent than "Search". I have re-compiled the LESS, and have that in the patch too.
Created attachment 30489 [details] [review] Attachment to Bug 12576 - search links need style id [SIGNED OFF] Bug 12576: Make the more/fewer options and "new search" links on the advanced search page look better. How to test: 1/ Visit /cgi-bin/koha/opac-search.pl 2/ Assert the "More/Fewer options" buttons look like bootstrap default buttons 3/ Assert the "New search" button looks like a bootstrap default button 4/ Assert the visited colour of the buttons remains #333 (light grey) 5/ Assert the buttons still function as expected Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
All worked as expected and looks great
Created attachment 31071 [details] [review] Bug 12576: Make the more/fewer options and "new search" links on the advanced search page look better. How to test: 1/ Visit /cgi-bin/koha/opac-search.pl 2/ Assert the "More/Fewer options" buttons look like bootstrap default buttons 3/ Assert the "New search" button looks like a bootstrap default button 4/ Assert the visited colour of the buttons remains #333 (light grey) 5/ Assert the buttons still function as expected Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 31072 [details] [review] Bug 12576: Update opac.css file This file had a conflict. I prefer to put the changes in a specific commit. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 31073 [details] [review] Bug 12576: Update opac.css file This file had a conflict. I prefer to put the changes in a specific commit. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
(In reply to Jonathan Druart from comment #5) > Created attachment 31072 [details] [review] [review] I forgot the --compress flag
Patches pushed to master. Thanks Sean!