durban77 opened a new pull request, #36273: URL: https://github.com/apache/superset/pull/36273
### SUMMARY DrillDetails uses fixed 50 for paging. The options on the UI allows the default 5/15/25/50/100 selection, that caused misbehaviour with other than 50 per page settings. The easiest way to fix this is to remove the possibility to change the pageSize. My approach is tho set the pageSizeOptions to overwrite the default values with an only 50 per page option. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF AFTER <img width="758" height="542" alt="image" src="https://github.com/user-attachments/assets/fbe928ce-e970-48d8-8f01-d292099c3e48"; /> there is only the 50/page option remained ### TESTING INSTRUCTIONS - You'll need to enable the drill to details functionality. - Right click on a chart with data and select drill to details option. - The appearing table must have a paging part on the bottom and a page size selector listbox on the right. - Before the modification, the options include 5/15/25/50/100 per page options. - After the modification, only 50 per page option remains. ### ADDITIONAL INFORMATION - [x] Has associated issue: #36267 - [ ] Required feature flags: - [x] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
