Summary: | Cannot scroll EANs when clicking 'Create EDIFACT order' in a basket | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Acquisitions | Assignee: | Martin Renvoize (ashimema) <martin.renvoize> |
Status: | Pushed to main --- | QA Contact: | Paul Derscheid <me> |
Severity: | major | ||
Priority: | P5 - low | CC: | lucas, martin.renvoize, oleonard |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
25.05.00
|
Circulation function: | |
Attachments: |
Bug 39754: Enable fixed height and overflow on EDIFACT dropdown
Bug 39754: Enable fixed height and overflow on EDIFACT dropdown Bug 39754: Enable fixed height and overflow on EDIFACT dropdown Bug 39754: Enable fixed height and overflow on EDIFACT dropdown |
Description
Nick Clemens (kidclamp)
2025-04-26 15:11:17 UTC
Created attachment 181578 [details] [review] Bug 39754: Enable fixed height and overflow on EDIFACT dropdown This patch updates the 'Create EDIFACT order' menu dropdown for cases where multiple EANs are available for the library. We set the dropdown to a max-height of 75% of the window height and then we enable y scrolling and hide the x scrollbar. We could perhaps improve this by adding an additional 'scrollable-dropdown' class to our stylesheets that set's the dropdown height and applies the two utility classes. But I'm not sure how often this is an problem so how much we want to do this in the upstream CSS Created attachment 181591 [details] [review] Bug 39754: Enable fixed height and overflow on EDIFACT dropdown This patch updates the 'Create EDIFACT order' menu dropdown for cases where multiple EANs are available for the library. We set the dropdown to a max-height of 75% of the window height and then we enable y scrolling and hide the x scrollbar. Signed-off-by: Nick Clemens <nick@bywatersolutions.com> I think there is a typo in the class name `overflow-y-autoi`. Created attachment 182400 [details] [review] Bug 39754: Enable fixed height and overflow on EDIFACT dropdown This patch updates the 'Create EDIFACT order' menu dropdown for cases where multiple EANs are available for the library. We set the dropdown to a max-height of 75% of the window height and then we enable y scrolling and hide the x scrollbar. Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Created attachment 182677 [details] [review] Bug 39754: Enable fixed height and overflow on EDIFACT dropdown This patch updates the 'Create EDIFACT order' menu dropdown for cases where multiple EANs are available for the library. We set the dropdown to a max-height of 75% of the window height and then we enable y scrolling and hide the x scrollbar. Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> Pushed for 25.05! Well done everyone, thank you! |