[🚀settings]: ability to hide settings tab when no active settings inside

[ala-n]

When multiple snippets used inside one UIP instance, there is a valid possibility of hiding all the settings. It is requested to allow users to get rid of empty settings container visually.

Suggesting to set up a marker on the inactive settings container to hide it.

[fshovchko]

@yadamskaya, @ala-n While working on implementation I ran into the following problem. There's already existing marker inactive, which indicates if all current setting items are inactive or not. Instead of implementing something on top, we can already use it to know when to hide the panel. But, it's being set with delay, since the listener is debounced by 100ms, which doesn't look good on changes. https://github.com/exadel-inc/ui-playground/blob/main/src/plugins/settings/settings.tsx#L109 What should we do about this? Is this debounce necessary? Maybe we can bypass the issue with animations?

[ala-n]

Oh, To be honest, I don't remember why we added a long delay for the marker attribute ) (Mb @yadamskaya remembers something regarding that point) Probably we can reduce it for formal 10s, replace with microtask or even get rid of it at all.

[yadamskaya]

@fshovchko @ala-n Yes, its true, marker inactive existis and indicates if all current setting items are inactive or not. We can use it for our purpose to know when to hide the settings panel.

I guess we can get rid of existing debounce, as I remember it was added before inavtive marker was implemented and I don't think there's any reason to use it. I don't remember why we needed it either :)

[yadamskaya]

:tada: This issue has been resolved in version 2.1.0-beta.5 :tada:

The release is available on:

• npm package (@beta dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: