Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AnalyticalTable: infinite scrolling with small chunk size does not allow to see all the data #5327

Closed
1 task done
cmealares opened this issue Dec 6, 2023 · 1 comment
Closed
1 task done
Labels
feature-request New feature or request wontfix This will not be worked on

Comments

@cmealares
Copy link

Describe the bug

When infinite scrolling uses a small chunk size, the table does not display the scrollbar.
Thus, the user cannot scroll down and can never see all the data.

Isolated Example

https://codesandbox.io/s/interesting-kepler-r3gy8y?file=/src/App.js

Reproduction steps

  1. Scroll down to get the next chunks of data. You can't because there is no scrollbar.

If you modify the code to use a bigger chunksize (say 50), then the scrollbar is visible again.

Expected Behaviour

The user must be able to access all the data.

Suggestion:
Force the scrollbar to be visible when infiniteScroll is true. And trigger onLoadMore() when the user tries to scroll down.

Screenshots or Videos

No response

UI5 Web Components for React Version

1.16.0

UI5 Web Components Version

1.14.0

Browser

Chrome

Operating System

No response

Additional Context

Organization: SAP

Independently of the chunk size, This issue can also happen with property isTreeTable= true:

  • Load a big chunk of data that is all under the same root
  • When this root node is expanded, the user can scroll down and see the rest of the data
  • But if the user collapses the first table node, then the scrollbar disappears and he cannot access the rest of the data.

Relevant log output

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@cmealares cmealares added the bug Something isn't working label Dec 6, 2023
@Lukas742 Lukas742 added feature-request New feature or request and removed bug Something isn't working labels Dec 7, 2023
@Lukas742
Copy link
Contributor

Lukas742 commented Dec 7, 2023

Hi @cmealares

unfortunately we have to reject this feature request for the following reasons:

  • As far as I know, no Sap/OpenUI5 table supports this option either
  • The implementation would feel weird, we would either have to add empty rows at the bottom of the table, which would then look strange when updating, or increase the height of the table body to make it scrollable, which would look weird when scrolling.

In my opinion, it would be better to use a "load more" button below the table in this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or request wontfix This will not be worked on
Projects
Status: 🆕 New
Development

No branches or pull requests

2 participants