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: hiding other folders on scroll #7

Conversation

joaoxmb
Copy link
Contributor

@joaoxmb joaoxmb commented Nov 28, 2023

Fixed the hiding of other folders unrelated to the one configured to be removed by the plugin, see the bug and the solution I took for it in this video

This bug is related to the styling applied to hide the folder in the DOM, specifically in the display: none; tag applied to hide it. The solution to this problem was to change display: none; to height: 0;overflow: hidden;.

@joaoxmb joaoxmb force-pushed the fix/hiding-other-folders-on-scroll branch from 8fb0da2 to 423576a Compare November 28, 2023 16:04
@joaoxmb joaoxmb force-pushed the fix/hiding-other-folders-on-scroll branch from 423576a to 8c0ba92 Compare November 28, 2023 16:05
@JonasDoesThings
Copy link
Owner

Hey, that's great. Thanks so much for your contribution! :)
I had problems reproducing this issue locally previously.

I will test it and if everything seems ok release a update with your PR later this day 👍

@JonasDoesThings
Copy link
Owner

Just a small change I pushed: instead of setting the whole cssText property, it now only sets the height & overflow properties. This is to decrease the (unlikely) risk of collisions with other plugins or themes that change css on the same element. Besides that, it looks good to me

@JonasDoesThings JonasDoesThings merged commit 5e910f1 into JonasDoesThings:master Nov 28, 2023
@JonasDoesThings
Copy link
Owner

Released in v1.2.1 🥳

@joaoxmb joaoxmb deleted the fix/hiding-other-folders-on-scroll branch November 29, 2023 03:15
@joaoxmb
Copy link
Contributor Author

joaoxmb commented Nov 29, 2023

I'm very happy to be able to contribute to something that really makes a difference to me. Now I can continue using it without any problems 🥳

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