There are some discrepancies: * Tags vs. Tag moderation * Barcode image generator and Quick spine label creator are switched * Upload any file vs. Upload (maybe use Uploads on both?) * With no installed tool plugin, the entry shows in sidebar, but not on the tools start page (conditions should match) * Cash management section is missing completely
Created attachment 140995 [details] [review] Bug 31402: Update tools sidebar to match tools start page This patch updates the tools sidebar so that it better matches the tools home page: - Sidebar menu should say Tags instead of Tag moderation - Barcode image generator and Quick spine label creator links reversed. - Upload any file -> Uploads - Cash management section is added to sidebar using brief text This patch does not make changes to the way plugins are displayed in the sidebar. In order for the sidebar to correctly display according to the presence of tool plugins I think some kind of global variable would need to be set.
I can confirm that the patch works in the sandbox except for one thing: "- Cash management section is added to sidebar using brief text" But then, I cannot see anything relating to a cash management section on the Tools home either - maybe there is a system preference that has to be set in order to see this? All the other points - Sidebar menu should say Tags instead of Tag moderation - Barcode image generator and Quick spine label creator links reversed. - Upload any file -> Uploads are there as expected.
You probably need to set UseCashRegisters :)
Thanks, Katrin. I now tried with UseCashRegisters activated, and there is indeed the Cash management section both in the Tools home and in the sidebar. In Tools home there shouldn't probably be the "for" after "Transaction history" (which makes it "Transaction history for" while in the sidebar it reads "Transaction history").
I think the 'for' appears when you haven't selected a cash register in your profile. Could certainly be improved in another bug.
I thought the names on the Tools home and the sidebar should be identical -? Anyhow, do you think the bug can be signed off now?
I think that was meant by 'brief text'. After the 'for' the library should appear, when a cash register is selected, but we probably should not display the for when the library is not set and the sentence will be incomplete. Leaving the 'for ...' off in the toolbar to make it a little shorter might be ok.
OK, well, this feature isn't exactly relevant for us, so I don't really care all that much. The patch does what it should in the sandbox, as I described, so it should be OK. Please sign off if you think fit - Thanks :-)
Hi Anke, hope that didn't come across wrong - I had stumbled on the lonely 'for' before as well. Do you not want to sign-off yourself?
Yes, I can do it, just wasn't too sure if this would be OK. Thanks for your help :-).
Created attachment 141187 [details] [review] Bug 31402: Update tools sidebar to match tools start page This patch updates the tools sidebar so that it better matches the tools home page: - Sidebar menu should say Tags instead of Tag moderation - Barcode image generator and Quick spine label creator links reversed. - Upload any file -> Uploads - Cash management section is added to sidebar using brief text This patch does not make changes to the way plugins are displayed in the sidebar. In order for the sidebar to correctly display according to the presence of tool plugins I think some kind of global variable would need to be set. Signed-off-by: Anke Bruns <anke.bruns@gwdg.de> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 22.11. Nice work everyone, thanks!