I've had pointed out to me that the search boxes in the head of the staff interface in unnecessarily small. For catalog search it could even shape the expectation of how many search terms is expected and unconsciously steer librarians towards broader searches. Since screens is often bigger these days and the area right of the search box is unused it seems a bit conservative to go with "size 40". I suggest that we make the container float up to some max size and then let the individual search boxes be for example size 60 but also have the CSS rule of "width:100%" to resize with the container. That way it will use the space better, scale up and down and still not become ridiculously huge on a 27".
Changes here may cause a merge conflict with the signed off 14189 since some of the same files must be updated.
Owen - we'll gladly let you write this patch. But give us a shout if not since Eivin is ready to write it otherwise.
Got an update from Cait that template bugs are auto assigned to Owen so we're going for this one ourselves :)
Created attachment 43572 [details] [review] Bug 14190: Scale search boxes in staff head In this simple patch the width gets controlled by CSS and it becomes easy to set it in just one place. The exact width may need to be discussed but I thought 30em made it look ok on my screen. With this patch we get the wider search boxes with a very small change and another bug may be created if needed to do a bigger refactor and remove stuff not needed in the html and perhaps do this in a modern/responsive way. (Preferrably a UX/GUI developer will do this) To test: Apply the patch and reload the staff client. Now the search box is a bit wider and the border etc has changed with it. Some places with more than one search box like "Serials" will not be affected.
Created attachment 43574 [details] [review] Bug 14190: Scale search boxes in staff head In this simple patch the width gets controlled by CSS and it becomes easy to set it in just one place. The exact width may need to be discussed but I thought 30em made it look ok on my screen. With this patch we get the wider search boxes with a very small change and another bug may be created if needed to do a bigger refactor and remove stuff not needed in the html and perhaps do this in a modern/responsive way. (Preferrably a UX/GUI developer will do this) To test: Apply the patch and reload the staff client. Now the search box is a bit wider and the border etc has changed with it. Some places with more than one search box like "Serials" will not be affected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 43945 [details] [review] [PASSED QA] Bug 14190: Scale search boxes in staff head In this simple patch the width gets controlled by CSS and it becomes easy to set it in just one place. The exact width may need to be discussed but I thought 30em made it look ok on my screen. With this patch we get the wider search boxes with a very small change and another bug may be created if needed to do a bigger refactor and remove stuff not needed in the html and perhaps do this in a modern/responsive way. (Preferrably a UX/GUI developer will do this) To test: Apply the patch and reload the staff client. Now the search box is a bit wider and the border etc has changed with it. Some places with more than one search box like "Serials" will not be affected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
I think this is really a question of taste somehow. I am passing this, as it can be easily changed back with a line in IntranetUserCSS and we can see how people like it :)
Patch pushed to master. Thanks Eivin and congratulations for your commit!