Bug 31705 - Option for ILL "Manage request" button to appear on left of the screen to improve accessibility
Summary: Option for ILL "Manage request" button to appear on left of the screen to imp...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: ILL (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Pedro Amorim
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-10-07 12:09 UTC by Jeremy Evans
Modified: 2023-12-28 20:43 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility (1.57 KB, patch)
2023-02-13 14:05 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility (1.57 KB, patch)
2023-02-22 10:27 UTC, Pedro Amorim
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Evans 2022-10-07 12:09:35 UTC
To improve accessibility and reduce excessive scrolling we would like an option for the "Manage Request" button on the list of ILL requests to appear on the left of the screen. It currently displays on the far right of the screen and normally requires scrolling beyond the columns displaying in the table for each request to be processed.
Comment 1 Katrin Fischer 2022-10-07 12:23:14 UTC
I'd like that too, not sure if it needs to be optional... maybe because we do have the button on the right usually,b ut with this wide table it does not work well.
Comment 2 Owen Leonard 2023-02-13 13:47:02 UTC
The simplest solution is to make some column in the table a link to the "Manage request" page. Even better would be to be more selective about which columns we show by default. Is it necessary to see them all in this view?
Comment 3 Katrin Fischer 2023-02-13 13:51:10 UTC
You can shorten the table with column configuration. As the backends might use different columns, I think having the table configuration is probably the best solution. 

What about linking the request ID in the first column? That seems like a logical choice?
Comment 4 Pedro Amorim 2023-02-13 13:58:03 UTC
(In reply to Owen Leonard from comment #2)
> The simplest solution is to make some column in the table a link to the
> "Manage request" page. Even better would be to be more selective about which
> columns we show by default. Is it necessary to see them all in this view?

I agree, the ID column looks to be ideal for this as will always have a value and would go in line with user's expectation, i.e. clicking on the ID takes the user to the details page.
Comment 5 Pedro Amorim 2023-02-13 14:05:38 UTC
Created attachment 146583 [details] [review]
Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility

This patch adds a link to the illview details page to the content in the ID column, making it accessible at the beginning of the row instead of having to scroll horizontally to find the "Manage request" button if all columns are being displayed.
Comment 6 Katrin Fischer 2023-02-20 22:37:30 UTC
I am sorry, but this doesn't apply:

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility
error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/js/ill-list-table.js).
error: could not build fake ancestor

Can you rebase your patch to master please?
Comment 7 Pedro Amorim 2023-02-22 10:27:50 UTC
Created attachment 147117 [details] [review]
Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility

This patch adds a link to the illview details page to the content in the ID column, making it accessible at the beginning of the row instead of having to scroll horizontally to find the "Manage request" button if all columns are being displayed.
Comment 8 Pedro Amorim 2023-02-22 10:30:20 UTC
Katrin hi, thank you!
I had accidentally done this on top of bug 30719. Should apply now.
I've updated the dependencies and plan to rebase 30719 on top of this soon.
Comment 9 Martin Renvoize 2023-05-05 13:33:01 UTC
I think this is actually already resolved with bug 22440 too.. feel free to switch to RESOLVED if that's the case Pedro
Comment 10 Pedro Amorim 2023-05-05 14:26:37 UTC
(In reply to Martin Renvoize from comment #9)
> I think this is actually already resolved with bug 22440 too.. feel free to
> switch to RESOLVED if that's the case Pedro

+1