revolist / revogrid

Powerful virtual data grid smartsheet with advanced customization. Best features from excel plus incredible performance 🔋
https://rv-grid.com
MIT License
2.74k stars 171 forks source link

Scrolling to focused cell not working when using Bootstrap 5 #349

Closed sven42 closed 2 months ago

sven42 commented 2 years ago

The default behavior seems to be that a focused cell is scrolled into the visible area. E.g. using cursor keys you can navigate quickly throw the table.

This does not work, when using Bootstrap 5 (Tested with 5.1 and 5.2, 5.2 was worse).

In context of https://github.com/revolist/revogrid/issues/344 I discovered, that part of the problem seems to be that some CSS settings are getting normalized by Bootstrap (and most probably by many other CSS-Solutions like Tailwind).

With the work around from https://github.com/revolist/revogrid/issues/344 in place, at least vertical scrolling is working again:

revo-grid [hidden] {
    display: initial !important;
}

However, I did not manage to find other settings which interfere with revo-grid and need to scroll horizontally with extra event handlers, which has other issues.

sven42 commented 2 years ago

Hello,

I just discovered that my scrolling issue was caused by a combination of using Bootstrap 5 and updating grid.columns when it was not needed.

So scrolling is working for me with the mentioned work around in #344

revolist commented 2 months ago

Marking as fixed.