Bug 30701 - On small screens, upload tool buttons cannot be clicked
Summary: On small screens, upload tool buttons cannot be clicked
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Aleisha Amohia
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-06 03:33 UTC by Aleisha Amohia
Modified: 2023-12-28 20:42 UTC (History)
0 users

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


Attachments
Bug 30701: Make Upload Tool buttons work on mobile-size screen (1.90 KB, patch)
2022-05-06 03:39 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30701: Make Upload Tool buttons work on mobile-size screen (1.95 KB, patch)
2022-05-06 20:27 UTC, David Nind
Details | Diff | Splinter Review
Screenshot of upload tool to display Step 3 of the test plan (50.27 KB, image/png)
2022-05-08 21:23 UTC, Aleisha Amohia
Details
Bug 30701: Make Upload Tool buttons work on mobile-size screen (2.00 KB, patch)
2022-05-13 14:38 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Aleisha Amohia 2022-05-06 03:33:52 UTC
The Upload tool can be accessed for Tools, and as a cataloguing plugin.

This was noticed when used for the plugin because it opened on a smaller window.

When the window is made smaller than 768 pixels, the 'Browse' and 'Upload' buttons in the 'Upload new file' section cannot be clicked.
Comment 1 Aleisha Amohia 2022-05-06 03:39:04 UTC
Created attachment 134670 [details] [review]
Bug 30701: Make Upload Tool buttons work on mobile-size screen

When the Upload tool is accessed on a smaller (mobile) screen (less than
768 pixels wide), the Upload and Browse buttons cannot be clicked. This
patch fixes that.

To test:
1) Go to Tools -> Upload
2) Make the screen narrower, either by opening your browser's Inspector
tool and using Responsive Design Mode, or by dragging in the edge of the
window to make it narrower.
3) Notice that when the screen becomes narrower than 768px, the sections
squash together and the padding between them is lost. Try to click on
the Browse and Upload buttons, notice you cannot.
4) Apply patch and refresh the Upload Tool
5) Ensure the screen is narrower than 768px. Confirm there is now
padding between the sections, and the buttons are clickable again.

Sponsored-by: Chartered Accountants Australia and New Zealand
Comment 2 Owen Leonard 2022-05-06 18:13:34 UTC
Could you attach a screenshot of how it looks before the patch? I'm not sure I'm seeing what you're seeing.
Comment 3 David Nind 2022-05-06 20:27:19 UTC
Created attachment 134762 [details] [review]
Bug 30701: Make Upload Tool buttons work on mobile-size screen

When the Upload tool is accessed on a smaller (mobile) screen (less than
768 pixels wide), the Upload and Browse buttons cannot be clicked. This
patch fixes that.

To test:
1) Go to Tools -> Upload
2) Make the screen narrower, either by opening your browser's Inspector
tool and using Responsive Design Mode, or by dragging in the edge of the
window to make it narrower.
3) Notice that when the screen becomes narrower than 768px, the sections
squash together and the padding between them is lost. Try to click on
the Browse and Upload buttons, notice you cannot.
4) Apply patch and refresh the Upload Tool
5) Ensure the screen is narrower than 768px. Confirm there is now
padding between the sections, and the buttons are clickable again.

Sponsored-by: Chartered Accountants Australia and New Zealand

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 Aleisha Amohia 2022-05-08 21:23:16 UTC
Created attachment 134769 [details]
Screenshot of upload tool to display Step 3 of the test plan

(In reply to Owen Leonard from comment #2)
> Could you attach a screenshot of how it looks before the patch? I'm not sure
> I'm seeing what you're seeing.

Screenshot attached. This describes Step 3 of the test plan. As you can see I've made the page smaller than 768px and the padding between the two sections is gone.
Comment 5 Katrin Fischer 2022-05-13 14:38:30 UTC
Created attachment 134975 [details] [review]
Bug 30701: Make Upload Tool buttons work on mobile-size screen

When the Upload tool is accessed on a smaller (mobile) screen (less than
768 pixels wide), the Upload and Browse buttons cannot be clicked. This
patch fixes that.

To test:
1) Go to Tools -> Upload
2) Make the screen narrower, either by opening your browser's Inspector
tool and using Responsive Design Mode, or by dragging in the edge of the
window to make it narrower.
3) Notice that when the screen becomes narrower than 768px, the sections
squash together and the padding between them is lost. Try to click on
the Browse and Upload buttons, notice you cannot.
4) Apply patch and refresh the Upload Tool
5) Ensure the screen is narrower than 768px. Confirm there is now
padding between the sections, and the buttons are clickable again.

Sponsored-by: Chartered Accountants Australia and New Zealand

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Comment 6 Fridolin Somers 2022-05-16 21:20:55 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄