mirror of
https://github.com/MonitorControl/MonitorControl.git
synced 2026-05-15 14:15:55 -06:00
[GH-ISSUE #536] Hide menu icon when no external monitor is connected #409
Labels
No labels
Status: Abandoned
arm64
beta
beta
bug
done
duplicate
enhancement
feedback needed from reporter
in progress
invalid
investigating
known Issue
monitor Issue
pull-request
translation
unable to reproduce
unreleased
x86
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/MonitorControl#409
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @mufumade on GitHub (Aug 24, 2021).
Original GitHub issue: https://github.com/MonitorControl/MonitorControl/issues/536
Originally assigned to: @waydabber on GitHub.
Before opening the issue, have you...?
Is your feature request related to a problem? Please describe
No
Describe the solution you'd like
Simple checkbox in the settings so the user can chose wether he likes to hide the menu icon when no external monitors are connected.
Describe alternatives you've considered
No alternatives.
Anything else?
No response
@waydabber commented on GitHub (Aug 25, 2021):
I am not sure about this one. There is already an option to hide the icon - selectively hiding/unhiding it based on which displays are present might create a confusion, but I understand why somebody would some prefer this one. An other thing is that now there are so many diverging minor requests on all fronts that solving all these will require an excessive amount of advanced preference items (I can like already count dozens) which makes me a bit cautious - maybe we should focus on the essentials only instead of creating an app that is customizable beyond a reasonable point? But from a technical point of view this request is absolutely doable.
I'll leave it open and let's see what others are saying - @JoniVR?
@JoniVR commented on GitHub (Aug 25, 2021):
I agree with @waydabber, this is a bit too much of a "special use case" that 99% of our users probably won't care about. It adds a bit extra maintenance burden and complicates the app a bit too much to be worth it imo.
@mufumade You're of course always free to fork the project (yay open source) and build it into your own fork, or (and I haven't tested it so idk if it's possible), you can try creating an automator action/script that launches/kills MonitorControl based on the amount of displays connected.
@mufumade commented on GitHub (Aug 25, 2021):
So my main point is that if you use your laptop on the go without any external monitors the menu space can be pretty limited. Especially on the smaller models. So to hide the menu item that has no use if no monitor is connected seems a good enough reason to me.
But thank you guys for your quick consideration :)
@waydabber commented on GitHub (Aug 25, 2021):
Hi @mufumade - I understand the reasoning. What I'd recommend is if you move the less important icon to the leftmost of the menu bar, macOS tends to hide them on a small screen if the currently open app has many menu items so there is not enough rooms left.
@waydabber commented on GitHub (Sep 8, 2021):
Hey all, I'll now close this feature request as both @JoniVR and I are concerned about making MonitorControl a bit unfocused with all kinds of feature requests now. If this comes up in the future or will see a demand for this feature we'll revisit it! Thank you!
@waydabber commented on GitHub (Sep 15, 2021):
Apparently this issue is requested by others as well.
https://github.com/MonitorControl/MonitorControl/discussions/590
Now that Advanced settings hides extra options throughout the preference panes even more arcane options could be added without confusing novice users so we can probably add this as well.
@waydabber commented on GitHub (Sep 15, 2021):
I added this one. See the updated screenshot here:
https://github.com/MonitorControl/MonitorControl/discussions/596