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

Hidden accordians - Permanetly unhide selctor in theme config #1698

Open
shoulders opened this issue Jul 21, 2024 · 9 comments
Open

Hidden accordians - Permanetly unhide selctor in theme config #1698

shoulders opened this issue Jul 21, 2024 · 9 comments

Comments

@shoulders
Copy link

SYSTEM INFORMATION
OS type and version Ubuntu Linux 22.04.4
Webmin version 2.111
Usermin version 2.010
Virtualmin version 7.20.1  
Theme version 21.10
Package updates All installed packages are up to date

I don't think I every did a GitHub issue for this, I have had a look in this repo, Webmin and Virtualmin and could not find one. I also was not sure if @iliajie add this feature.

the background

46019ee9f83d23cdbc3682f98526a5e70f1b70e4_2_689x478

The issue

Unless you hide any of the panels on your dashboard the hidden accordians selector will not be shown

I spent hours looking for this before I was willing to hide a panel I might need again

proposed solution

make the Hidden accordians selector in theme config visible all the time as there is no need for it to be hidden.

Additional

discussed here

https://forum.virtualmin.com/t/where-do-you-unhide-dashboard-panels/126925/5

@iliajie
Copy link
Collaborator

iliajie commented Jul 21, 2024

Hello,

For the latest release, I already made it clear in the alert where it can be un-hidden after it was hidden.

@shoulders
Copy link
Author

shoulders commented Jul 21, 2024

image

This is great but not what I meant, althought this is helpful. I meant always show Hidden accordions in theme config

image

Code

# Exclude hidden panels if none
if (!$theme_config{'settings_sysinfo_hidden_panels_user'}) {
push(@theme_settings_filter, 'settings_sysinfo_hidden_panels_user');
}

I am guessing this is the right code.

Remove the following which will prevent "hidden accordions` select thing from being hidden in theme config.

# Exclude hidden panels if none
if (!$theme_config{'settings_sysinfo_hidden_panels_user'}) {
    push(@theme_settings_filter, 'settings_sysinfo_hidden_panels_user');
}

@iliajie
Copy link
Collaborator

iliajie commented Jul 26, 2024

This is great but not what I meant, althought this is helpful. I meant always show Hidden accordions in theme config

Sure, makes sense, thanks! I'll take a look.

@shoulders
Copy link
Author

bump 😄

this is a nice easy one to fix

@iliajie
Copy link
Collaborator

iliajie commented Oct 1, 2024

Unless you hide any of the panels on your dashboard the hidden accordians selector will not be shown

I spent hours looking for this before I was willing to hide a panel I might need again

Why would you search for something that doesn’t exist?

I’m not sure displaying an empty select will be less confusing, but I did improve the message after the accordion was hidden, explaining where it can be found.

@shoulders
Copy link
Author

This is an easy one

  • before I hid the panels I wanted to make sure I knew where to unhide them.
  • I could not find where to unhide the panels, ironically because this feature was hidden.
  • As an end user this does not make sense that it is hidden and a few others on the forum had this issue, jimr thought the only way to restore the panels was to reset the the theme. I am sure there are other people and he has been using Webmin for years.
  • I have never used a software where a setting like this would be hidden like this.

@iliajie
Copy link
Collaborator

iliajie commented Oct 1, 2024

  • before I hid the panels I wanted to make sure I knew where to unhide them.

You can't really know what will happen until you actually hide the panel. :-)

I could not find where to unhide the panels, ironically because this feature was hidden.

You couldn’t find where it was hidden because we didn’t specify where to look. Now we do., e.g.:

image

As an end user this does not make sense that it is hidden and a few others on the forum had this issue, jimr thought the only way to restore the panels was to reset the the theme. I am sure there are other people and he has been using Webmin for years.

Ah, okay! Then in this case, we should always display a select with all available panels (whether selected or deselected), rather than showing an empty one. It's probably not an easy fix, but I’ll take a look!

I have never used a software where a setting like this would be hidden like this.

You actually have!! :--))

@shoulders
Copy link
Author

One additional thought.

When this is implemented, the visible panels should be highlighted in the list.

@iliajie
Copy link
Collaborator

iliajie commented Oct 1, 2024

When this is implemented, the visible panels should be highlighted in the list.

Yes, just it is now for disabled ones.

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

No branches or pull requests

2 participants