| Summary: | Improve visibility of place hold option when checking out an item already checked out | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Owen Leonard <oleonard> |
| Component: | Circulation | Assignee: | Aleisha Amohia <aleisha> |
| Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P3 | CC: | aleisha, gmcharlt, jonathan.druart, kyle.m.hall, veron |
| Version: | Main | ||
| Hardware: | PC | ||
| OS: | All | ||
| See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1983 | ||
| Change sponsored?: | Sponsored | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Screenshot of the proposed change
Bug 15681: Improve visibility of place hold option when checking out an item already checked out Screenshot of a test case |
||
|
Description
Owen Leonard
2016-01-27 16:29:13 UTC
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 (...) |