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

[BUG] Log table rows in the "New Discover" UI flicker after scrolling to the bottom #5512

Closed
gjd6640 opened this issue Nov 20, 2023 · 4 comments
Labels
bug Something isn't working discover for discover reinvent

Comments

@gjd6640
Copy link

gjd6640 commented Nov 20, 2023

Describe the bug

Log table rows in the "New Discover" UI flicker after scrolling to the bottom.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the Discover dashboard (leave the "New Discover" theme enabled)
  2. Adjust the search criteria and refresh such that multiple pages of logs are found
  3. Scroll down to the bottom of the list of log entries
  4. Observe: There should be links to multiple pages of results in the bottom right
  5. Observe: The log entries data in the table all now flicker between log data and blank

Expected behavior
Log lines in the table should not flicker. This makes them unusable.

OpenSearch Version

Unknown - I think as just a user I lack permission to get to this info

Dashboards Version

v2.10.0 (found this value in the '?' menu)

Plugins

Google Docs Offline
Symantec Extension
Axiad Portal Extension

Screenshots

rlr2

Host/Environment (please complete the following information):

  • OS: Windows 10 enterprise 10.0.19045 Build 19045
  • Chrome: 119.0.6045.124 (Official Build) (64-bit)
@gjd6640 gjd6640 added bug Something isn't working untriaged labels Nov 20, 2023
@ananzh ananzh added discover for discover reinvent and removed untriaged labels Nov 21, 2023
@ananzh ananzh self-assigned this Nov 21, 2023
@ananzh
Copy link
Member

ananzh commented Nov 21, 2023

Hey @gjd6640 thanks for opening this issue. I would like to reproduce it first in 2.10. Have 2 questions

  1. For Adjust the search criteria and refresh such that multiple pages of logs are found, what do you mean by adjust the search criteria? Is it add some search field in the side nav? add a query in the top nav?
  2. For Observe: There should be links to multiple pages of results in the bottom right, what are the links to multiple pages? you mean pagination?

Is there a way that you could help us add a screen recording and add it in the issue? and a clear screen shot for the expected links to multiple pages. This would help us quickly understand and resolve the issue. Thanks a lot for testing new Discover for the community!!!

@ananzh ananzh added the v2.12.0 label Nov 21, 2023
@gjd6640
Copy link
Author

gjd6640 commented Nov 27, 2023

Here's the filter criteria. I'm not really filtering for this example other than using the dev environment index. Note that while capturing this I noticed that depending on which columns I include in my results and what order I put them in the flicker problem sometimes goes away.

OpenSearchUI

Here's a video showing the flicker:

2023-11-27.09-56-46.mp4

Here's some performance data captured by the browser showing that a scroll event seems to be firing repeatedly:

OpenSearchUI_performanceAnalysis

@ananzh ananzh assigned abbyhu2000 and unassigned ananzh Dec 22, 2023
@mz118
Copy link

mz118 commented Jan 10, 2024

I saw similar behaviour in 2.11.0.

@ananzh
Copy link
Member

ananzh commented Jun 21, 2024

Performance is resolved in #6599
Close it for now. Let us if there is any other issues.

@ananzh ananzh closed this as completed Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working discover for discover reinvent
Projects
None yet
Development

No branches or pull requests

5 participants