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

System's context menu captures a scrolling of the table's container. #5846

Closed
swistach opened this issue Mar 7, 2019 · 4 comments
Closed

System's context menu captures a scrolling of the table's container. #5846

swistach opened this issue Mar 7, 2019 · 4 comments

Comments

@swistach
Copy link
Contributor

@swistach swistach commented Mar 7, 2019

Description

System's Context Menu captures a scrolling of the table's container - until you click on the table again.

Steps to reproduce

  1. Go to https://handsontable.com/docs/7.0.0/demo-scrolling.html
  2. Click RMB on A1.
  3. Press ESC to close context menu.
  4. Move the cursor outside of the table's container (on the right side or below)

Demo

https://handsontable.com/docs/7.0.0/demo-scrolling.html

Nov-15-2019 16-11-28

Your environment

  • Handsontable version: any
@aninde

This comment has been minimized.

Copy link

@aninde aninde commented Nov 15, 2019

This issue will be fixed in 7.3.0 v. I tested it on Chrome 78. Firefox 70, Safari 10.13, Microsoft Dev Edge, Opera.

Described here case is related with Chrome.
cointextMenu7 3 0

Case 2 Firefox
Don't release RMB
On previous version there it caused scrolling.
scroll-7 2 2ff

7.3.0 v. fixed it.
Firefox
scrolling-7 3ff

Chrome
scrolling-holding7 3chrome

Demo
7.3.0 v forked from scrolling demo https://jsfiddle.net/aninde/bz3uw64e/
7.3.0 v. iframe https://jsfiddle.net/jysznLxe/

@swistach swistach self-assigned this Nov 18, 2019
@swistach swistach added this to the October 2019 milestone Nov 18, 2019
@AMBudnik

This comment has been minimized.

Copy link
Contributor

@AMBudnik AMBudnik commented Nov 18, 2019

Fixed for Firefox 70 / Chrome 78

@AMBudnik

This comment has been minimized.

Copy link
Contributor

@AMBudnik AMBudnik commented Dec 6, 2019

After loading the latest build (6.12) issue is still fixed https://jsfiddle.net/AMBudnik/t8podj5z/

Tested on Catalina with Chrome 78, FF 70 and Safari.

@aninde

This comment has been minimized.

Copy link

@aninde aninde commented Dec 9, 2019

Confirmed, like @AMBudnik wrote it still works good during testing 7.3.0 v. (6.12) on Chrome 78, Firefox 70, Safari 13, Microsoft Dev Edge / Mac OS Catalina and latest IE, Edge, Edge Dev / Windows 10
https://jsfiddle.net/aninde/qmvLy7hn/
https://jsfiddle.net/aninde/1n2usx6h/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.