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

Add ability to hide panels in the tab stack #41

Open
mzso opened this issue Aug 29, 2016 · 7 comments
Open

Add ability to hide panels in the tab stack #41

mzso opened this issue Aug 29, 2016 · 7 comments

Comments

@mzso
Copy link

mzso commented Aug 29, 2016

Why not? It would be the easiest thing, just don't show the tab.

The unavailability of other item details options make sense (auto hide, locked, show caption, create left toggle area.) This doesn't.

Also I'm quite I could hide panels in tab stacks in older versions.

@reupen
Copy link
Owner

reupen commented Aug 30, 2016

Notionally, it is disabled because switching to another tab hides the other panels in the tab stack.

What scenario(s) have you wanted to hide the tab itself in but not remove it?

@mzso
Copy link
Author

mzso commented Aug 30, 2016

To deactivate a panel, but to not remove it. (To conserve limited space and maybe system resources) Sometimes I'm trying out different panels with a similar functionality. This time I just wanted to disable foo_jscript_panel with the "Last.fm Album Art Downloader" script which I couldn't get to work to my liking.

@reupen reupen changed the title Can't hide panels on tab stack. Add ability to hide panels in the tab stack Aug 30, 2016
@reupen
Copy link
Owner

reupen commented Aug 30, 2016

Sounds reasonable. I would argue it is a bit different from the current 'Hidden' option as in the vertical/horizontal splitters it just gives them a zero width/height (leaving the divider visible, which can be double-clicked on to make them visible again).

@mzso
Copy link
Author

mzso commented Aug 30, 2016

@reupen commented on 2016. aug. 30. 22:06 CEST:

Sounds reasonable. I would argue it is a bit different from the current 'Hidden' option as in the vertical/horizontal splitters it just gives them a zero width/height (leaving the divider visible, which can be double-clicked on to make them visible again).

I see. Well it always seemed a bit fishy the other splitters, because a margin or something remained so it wasn't exactly like if they were removed the remaining elements were a little offset.

Maybe in a similar vein, the panels on the horizontal/vertical splitters could also be removed when set to hidden? Or are the panels are doing their thing when hidden? (If yes, how is that useful? Panels provide some sort of visual function)

@reupen
Copy link
Owner

reupen commented Aug 30, 2016

I think the original request was for hiding and showing panels (or the original sidebar that was in early versions of CUI) by double-clicking on the divider. The setting was just a consequence of having that feature.

Anyway, it doesn't matter that much, just something to consider if this were to be implemented.

@ghost
Copy link

ghost commented Sep 11, 2016

I'm totally against this suggestion. It's quite desirable to have WSH panel mod/JScript panel instances doing non-UI related tasks in the background while hidden. If you have a problem with a given script, remove it. If you know what you're doing, scripts can be edited to prevent things happening when their height or width is zero. That should be enough.

@mzso
Copy link
Author

mzso commented Sep 11, 2016

@19379 commented on 2016. szept. 11. 22:07 CEST:

I'm totally against this suggestion. It's quite desirable to have WSH panel mod/JScript panel instances doing non-UI related tasks in the background while hidden. If you have a problem with a given script, remove it. If you know what you're doing, scripts can be edited to prevent things happening when their height or width is zero. That should be enough.

Well, it still can be hidden then. Although I don't see much point in having a panel that doesn't show anything.
If this is the case a separate disable option might make mores sense.

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

No branches or pull requests

2 participants