Created attachment 76214 [details] [review] Bug 20970: Reformat basket information on acquisitions basket page This patch reformats the display of information on the acquisitions basket page, hiding form controls until they are triggered. Some basket information is now in the second column in order to balance the two columns better. The "Library" label has been replaced with the more precise "Managing library." Unrelated change: Removed script type attributes. To test, apply the patch and view a basket in acquisitions. Test these processes: - Adding and removing managing users - Setting and changing managing library - Setting and changing basket group This looks way better than the grey buttons. I did have a couple of problems though 1) I had trouble applying the patch, I had to add manually basket.tt and then resume applying. 2) Once the patch is applied, if I click on "Add user", nothing happens, but if I remove the patch, and click on "Add user", the pop-up appears. 3) Same thing if I click on "Set library" or "Change basket group" And my other comment is purely personal, I love the way it looks, but it adds one click. To change a basket group, for example, you have to click on "Change basket group", then choose the basket group in the drop down, and save (4 clicks). Whereas now, you just choose the basket group in the drop down and save (3 clicks). It's not the end of the world, and I can absolutely live with one click more, but I just wanted to throw it out there. Created attachment 77498 [details] [review] Bug 20970: Reformat basket information on acquisitions basket page This patch reformats the display of information on the acquisitions basket page, hiding form controls until they are triggered. Some basket information is now in the second column in order to balance the two columns better. The "Library" label has been replaced with the more precise "Managing library." Unrelated change: Removed script type attributes. To test, apply the patch and view a basket in acquisitions. Test these processes: - Adding and removing managing users - Setting and changing managing library - Setting and changing basket group Thanks for testing Caroline. I proposed this change because I think the benefit of a compact and (I hope) more readable display outweighs the downside of an extra click. I've rebased the patch. Can you try again? The patch applies fine now. Everything works as advertised. That might be beyond your remodeling changes, but maybe we could ditch the "Save changes" button when adding/deleting users? It would save a click (hehe!), and I don't find it super intuitive to have to save once the user appears on the page. When deleting a user, I get that you need a confirmation so that people don't delete users by accident, but even then, I don't find "Save changes" intuitive. And also, if I make a change without saving, and leave the page, there is no warning that I'm leaving unsaved changes (like in the system preferences pages, for example). Created attachment 77521 [details] [review] Bug 20970: (follow-up) Reformat basket information on acquisitions basket page This patch adds auto-submit to the branch and basket group selection forms. To test, apply the patch and view a basket in acquisitions. Test the process of changing the managing library and basket group. In both cases, selecting an choice from the dropdown should automatically submit the form. Test as a user with and without permission to manage basket groups. (In reply to Caroline Cyr La Rose from comment #5) > The patch applies fine now. Everything works as advertised. > > That might be beyond your remodeling changes, but maybe we could ditch the > "Save changes" button when adding/deleting users? Good idea! I was able to make it so for the managing library and basket group selection. Created attachment 79172 [details] [review] Bug 20970: Reformat basket information on acquisitions basket page This patch reformats the display of information on the acquisitions basket page, hiding form controls until they are triggered. Some basket information is now in the second column in order to balance the two columns better. The "Library" label has been replaced with the more precise "Managing library." Unrelated change: Removed script type attributes. To test, apply the patch and view a basket in acquisitions. Test these processes: - Adding and removing managing users - Setting and changing managing library - Setting and changing basket group When IndependentBranches in on, it's not possible to choose the only available site for 'managing library' : the page never reload. Works fine for 'users' and 'basket group'. Created attachment 80354 [details] [review] Bug 20970: Reformat basket information on acquisitions basket page This patch reformats the display of information on the acquisitions basket page, hiding form controls until they are triggered. Some basket information is now in the second column in order to balance the two columns better. The "Library" label has been replaced with the more precise "Managing library." Unrelated change: Removed script type attributes. To test, apply the patch and view a basket in acquisitions. Test these processes: - Adding and removing managing users - Setting and changing managing library - Setting and changing basket group Created attachment 80954 [details] [review] Bug 20970: Reformat basket information on acquisitions basket page This patch reformats the display of information on the acquisitions basket page, hiding form controls until they are triggered. Some basket information is now in the second column in order to balance the two columns better. The "Library" label has been replaced with the more precise "Managing library." Unrelated change: Removed script type attributes. To test, apply the patch and view a basket in acquisitions. Test these processes: - Adding and removing managing users - Setting and changing managing library - Setting and changing basket group Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Created attachment 81028 [details] [review] Bug 20970: Reformat basket information on acquisitions basket page This patch reformats the display of information on the acquisitions basket page, hiding form controls until they are triggered. Some basket information is now in the second column in order to balance the two columns better. The "Library" label has been replaced with the more precise "Managing library." Unrelated change: Removed script type attributes. To test, apply the patch and view a basket in acquisitions. Test these processes: - Adding and removing managing users - Setting and changing managing library - Setting and changing basket group Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 81029 [details] [review] Bug 20970: Add preventDefault to avoid trailing # to be added to the url Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Awesome work all! Pushed to master for 18.11 Enhancement, will not be backported to 18.05.x series. |
Created attachment 76213 [details] GIF showing new forms interaction I'd like to propose a couple of minor changes to the way information and forms are displayed on the acquisitions basket view page: - Hide form controls until the user clicks a link - Move some information into the second column to make it a little more balanced.