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

Duplicated scroll behaviour when using TruncatedFormat Table2 cells #7057

Open
berryde opened this issue Nov 11, 2024 · 1 comment
Open

Duplicated scroll behaviour when using TruncatedFormat Table2 cells #7057

berryde opened this issue Nov 11, 2024 · 1 comment
Assignees

Comments

@berryde
Copy link
Contributor

berryde commented Nov 11, 2024

Environment

  • Package version(s): "@blueprintjs/core": "^5.14.0", "@blueprintjs/table": "^5.2.0"
  • Operating System: Mac OS 14.7.1 (23H222)
  • Browser name and version: Chrome Version 1.53.21 Chromium: 130.0.6723.117

Steps to reproduce

This issue can be reproduced using the documentation example here.

  1. Select the "…" icon on a TruncatedFormat cell to view the cell contents in a popover.
  2. Attempt to scroll the cell contents (it may be necessary to add some cell contents via 'Inspect element' to trigger the overflow).

Actual behaviour

The cell contents in the popover should scroll correctly, however the issue is that the table also scrolls which can cause the popover to be scrolled out of view.

A similar issue was also reported in #5469 and #2651 – this seems to have been an issue for several years but since I also noticed the issue I thought it would be worthwhile to initiate a new discussion for visibility.

Expected behaviour

When a TruncatedFormat popover is open, scrolling within the popover should not also scroll the table.

Possible solution

TruncatedFormat popovers should not propagate scroll events to parents. I am not sure if this would definitely fix the issue.

@JML321
Copy link

JML321 commented Nov 20, 2024

@jscheiny, I hope you don't mind that I took a stab at this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants