mirror of
https://github.com/MonitorControl/MonitorControl.git
synced 2026-05-15 14:15:55 -06:00
[PR #1716] [MERGED] Add ability to command-drag out status item to remove #1277
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#1277
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?
📋 Pull Request Information
Original PR: https://github.com/MonitorControl/MonitorControl/pull/1716
Author: @pmarell
Created: 1/5/2025
Status: ✅ Merged
Merged: 1/20/2025
Merged by: @waydabber
Base:
main← Head:main📝 Commits (1)
51429f2Add ability to drag out status item to remove📊 Changes
3 files changed (+33 additions, -5 deletions)
View changed files
📝
MonitorControl/Support/AppDelegate.swift(+22 -3)📝
MonitorControl/Support/MenuHandler.swift(+2 -2)📝
MonitorControl/View Controllers/Preferences/MenuslidersPrefsViewController.swift(+9 -0)📄 Description
A small change which adds the ability to command-drag the status item out of the menu bar to remove it, as a shortcut.
When this happens, the "Menu Icon" setting updates to "Always hide" automatically. With this behaviour, there were also some cases where that setting would get updated inadvertently from other changes to the status item's visibility. A
updateStatusItemVisibilitymethod was added to reliably detect whether the status item's visibility was due to the user dragging out the item, and not from something else.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.