Created attachment 47356 [details] Screenshot of the proposed change Bug 1983 adds a link to place a hold on an item instead of checking it out when the item is already checked out. Sorry to be late to the party, but I have a couple of issues with the way it works: 1. Saying "click here" is almost always redundant, and doesn't represent the different ways in which users might interact with links (touchscreen? keyboard navigation?). 2. I'm concerned that the link is not very visible and doesn't well represent what is going to happen next. I think the link is a third option with equal weight as the confirm and cancel buttons and should be represented as a button even if the button text makes it big. I would love to hear comments.
Yes, I completely agree. I have asked when QAing bug 1983, and was not really convinced by the answer. But let it pass anyway.
Created attachment 47880 [details] [review] Bug 15681: Improve visibility of place hold option when checking out an item already checked out Have converted icons to font-awesome! To test: 1) Check out an item to a user 2) Attempt to check out the same item (ie same barcode) to another user 3) Confirm that message box says 'Please confirm checkout' with 3 newly styled buttons 4) Confirm 'cancel checkout and place a hold' button is styled the same as the other two buttons and clicking it takes you to page to place holds (reserves.pl) 5) Confirm behaviour of other two buttons works as expected also Sponsored-by: Catalyst IT
Created attachment 47891 [details] Screenshot of a test case In testing this I randomly happened to pick an item which was both checked out and on hold for someone, so this dialog got pretty complicated. Currently buttons in dialogs have a special style which is different from other buttons. As you can see the patch doesn't modify them all. I would prefer to not use the Bootstrap style buttons here and keep the existing dialog button style. I think if we want to change the style of dialog buttons we should do it across the board. I'm beginning to do so--at least for the icons--with Bug 15785.
(In reply to Owen Leonard from comment #3) > Created attachment 47891 [details] > Screenshot of a test case > > In testing this I randomly happened to pick an item which was both checked > out and on hold for someone, so this dialog got pretty complicated. > > Currently buttons in dialogs have a special style which is different from > other buttons. As you can see the patch doesn't modify them all. I would > prefer to not use the Bootstrap style buttons here and keep the existing > dialog button style. I think if we want to change the style of dialog > buttons we should do it across the board. > > I'm beginning to do so--at least for the icons--with Bug 15785. Ah I see! I think we should be changing all of the buttons and icons to be consistent with the buttons all over the site. Should I make a patch for circ templates like you have for acquisitions and add it to your bug or just make a new bug for each module?
(In reply to Aleisha Amohia from comment #4) > Ah I see! I think we should be changing all of the buttons and icons to be > consistent with the buttons all over the site. I see a value in having (slightly) different button styles for different areas, but this isn't a rule that anyone has discussed or agreed on. For instance: Toolbars get one button style, dialogs get another, everything else a third. This obviously isn't something we're being consistent about now because many new template changes are using Bootstrap-style buttons by default wherever they are. I'm interested in hearing your (and anyone else's) opinion about that idea. Koha really needs a style guide. More work, though, boo. > Should I make a patch for > circ templates like you have for acquisitions and add it to your bug or just > make a new bug for each module? I think a new bug for each module makes sense so that it's easy to keep track of which patches depend on which.
(In reply to Owen Leonard from comment #5) (...) > I see a value in having (slightly) different button styles for different > areas, but this isn't a rule that anyone has discussed or agreed on. For > instance: Toolbars get one button style, dialogs get another, everything > else a third. This obviously isn't something we're being consistent about > now because many new template changes are using Bootstrap-style buttons by > default wherever they are. +1 (...) > Koha really needs a style guide. +1 (...)
Will fix this in patch for Bug 15821.
*** This bug has been marked as a duplicate of bug 15821 ***