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

Fixed an issue where first level folders were auto-expanded. #5355

Merged

Conversation

valadas
Copy link
Contributor

@valadas valadas commented Oct 23, 2022

In #5351 a change was implemented to get the root folders according to the settings. Because of this a user is now always on some sort of root folder instead of the root folders not being part of any and having a virtual "root" ListItem. So before that change we were auto-expending that root item but now all direct children are open by default and it does not look right. This PR solves that by not expanding automatically the first level in the tree.

Before:
image

After:
image

In dnnsoftware#5351 a change was implemented to get the root folders according to the settings. Because of this a user is now always on some sort of root folder instead of the root folders not being part of any and having a virtual "root" ListItem. So before that change we were auto-expending that root item but now all direct children are open by default and it does not look right. This PR solves that by not expanding automatically the first level in the tree.
@valadas valadas added this to the 9.11.1 milestone Oct 23, 2022
@david-poindexter david-poindexter merged commit 24a9a57 into dnnsoftware:develop Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants