Bug 31705

Summary: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility
Product: Koha Reporter: Jeremy Evans <jeremy.evans>
Component: ILLAssignee: Pedro Amorim <pedro.amorim>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P5 - low CC: lucas, martin.renvoize, pedro.amorim
Version: unspecified   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22440
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility
Bug 31705: Option for ILL "Manage request" button to appear on left of the screen to improve accessibility

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 (ashimema) 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