Skip to content

Add scrollToIndex #53

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

Merged
merged 6 commits into from
Aug 4, 2018
Merged

Add scrollToIndex #53

merged 6 commits into from
Aug 4, 2018

Conversation

Avivhdr
Copy link
Contributor

@Avivhdr Avivhdr commented Jul 27, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Jul 27, 2018

Codecov Report

Merging #53 into master will increase coverage by 0.11%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   96.52%   96.63%   +0.11%     
==========================================
  Files          15       15              
  Lines         115      119       +4     
==========================================
+ Hits          111      115       +4     
  Misses          4        4
Impacted Files Coverage Δ
src/TreeContainer.js 100% <100%> (ø) ⬆️
src/Tree.js 83.33% <100%> (ø) ⬆️
src/selectors/nodes.js 96.42% <100%> (+0.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2509ac...5bb56d7. Read the comment docs.

@coveralls
Copy link

coveralls commented Jul 27, 2018

Coverage Status

Coverage increased (+0.1%) to 94.857% when pulling 5bb56d7 on Avivhdr:master into e2509ac on diogofcunha:master.

Copy link
Owner

@diogofcunha diogofcunha left a comment

Choose a reason for hiding this comment

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

Nice job Aviv.

Can you please just add the scrollToId prop to the typescript definitions? They live in https://github.com/diogofcunha/react-virtualized-tree/blob/master/index.d.ts#L32.

In case you are not familiar with typescript, the syntax is

scrollToId?: number

@Avivhdr
Copy link
Contributor Author

Avivhdr commented Aug 4, 2018

@diogofcunha
I've made the change.

@diogofcunha diogofcunha merged commit fa7af12 into diogofcunha:master Aug 4, 2018
@diogofcunha
Copy link
Owner

🎉 This PR is included in version 2.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants