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 horizontal scrolling #80

Merged
merged 2 commits into from
Jul 27, 2022
Merged

Fix horizontal scrolling #80

merged 2 commits into from
Jul 27, 2022

Conversation

jcshott
Copy link
Contributor

@jcshott jcshott commented Jul 27, 2022

Update react-data-grid to version v5x (which appears to fix no horizontal scrolling)

there are deprecation warnings that show up and this solves it: adazzle/react-data-grid#1403 (comment)

I didn't want to upgrade all the way to v7x (which fixed the warnings) as it doesn't seem stable and could open a pandora's box of problems. one version upgrade seems to be ok.

bp-scroll

Update react-data-grid to version v5x
@@ -8,17 +8,13 @@ export default class RowRenderer extends React.Component {
idx: PropTypes.number.isRequired
};

setScrollLeft = (scrollBy) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this didn't appear to be used and was just confusing.

Copy link
Contributor

@JonathanAquino-NextRoll JonathanAquino-NextRoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@JonathanAquino-NextRoll
Copy link
Contributor

Would you be able to link to the react-data-grid PR that fixes horizontal scrolling or a note in the version history on this, if you have it handy?

@jcshott
Copy link
Contributor Author

jcshott commented Jul 27, 2022

I'll see if I can find it - after a lot of searching (v4 didn't have a lot of info), I just tried updating to see if it worked - figuring a 4 -> 5 would minimize other potential issues (since we don't really use a lot of features) and it seemed to fix it. so I just ran with it :) but I might be able to figure out what was fixed and link here.

Edit: I believe it's version 5.0.2 that fixes horizontal scrolling there were a couple updates that addressed issues

@jcshott jcshott merged commit 1e70c71 into master Jul 27, 2022
@jcshott jcshott deleted the fix-horizontal-scrolling branch July 27, 2022 19:10
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

Successfully merging this pull request may close these issues.

2 participants