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

fix autoscroll on long columns #4479

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

azmy60
Copy link
Contributor

@azmy60 azmy60 commented Apr 22, 2024

Tabulator doesn't scroll to a long column correctly.

To replicate: Use keyboard to navigate to a long column
Expected: It should scroll to the left side of the cell
longcolumn2

Actual: It scrolls to the right side of the cell
longcolumn

@olifolkerd
Copy link
Owner

Hey @azmy60

Same as #4478 on the getters.

Also looks like this now has conflicts having merged in some of your other tweaks.

Cheers

Oli :)

@olifolkerd olifolkerd added the PR Needs Work The PR is a good idea, but needs some updates before it can be merged label Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR Needs Work The PR is a good idea, but needs some updates before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants