Skip to content
This repository has been archived by the owner on Jun 17, 2022. It is now read-only.

Scroll to groups #24

Merged
merged 2 commits into from
May 14, 2020
Merged

Scroll to groups #24

merged 2 commits into from
May 14, 2020

Conversation

MiguelLopez6
Copy link
Contributor

Added a missing id to the groups so they can be found by id.
Also, made the actual scroll perform after a timeout promise, so it gets the right offset of the tab group after the cycle has completed and the group is visible in the DOM.

Perform actual scroll to group after current tab has been updated and is visible in the DOM. In this way it can get the actual offset of the group and scroll after changing the tab. Previously it changed tab but didn't scroll down to group as it return offset = 0; The getScrollPositionFor returns a timeout promise, with the group offset after it has been made visible.
@MiguelLopez6
Copy link
Contributor Author

Hi,
I found these issues while using the package on Umbraco 8.6.1.
Thanks for the package, the group division is really nice, especially for doctypes with a large number of properties.

@skttl
Copy link
Owner

skttl commented May 14, 2020

Works like a charm thanks :)

Funny thing is, I do actually have the id on my local environment, just forgot to push it 😳

@kipusoep
Copy link

Lol! 😂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants