Panel wächst in die Breite, Menüleiste wird schmaler
Das Raster ist weg. Widgets stehen nebeneinander, alle gleich groß, in der Reihenfolge aus den Einstellungen. Das Panel sitzt am oberen Rand — nach unten zu wachsen bedeckt den Bildschirm, nach rechts und links legt es sich in den ohnehin leeren Streifen neben der Notch. Eine zweite Reihe entsteht erst, wenn der Bildschirm keine weitere Kachel mehr hergibt. Die Größenauswahl fällt damit weg, und das ist kein Verlust: ein 2×1 neben einem 2×2 lässt oben rechts eine Lücke, die niemand füllen kann und die aussieht wie ein Fehler. Genau daher kam auch die Überlappung. Ein Test prüft jetzt für ein bis zwölf Kacheln, dass sich keine zwei schneiden und alle vollständig im Panel liegen — außerhalb heißt oben: hinter der Notch. Kacheln werden zusätzlich beschnitten, damit überquellender Inhalt nicht über den Nachbarn zeichnet. Der Kalender wählt jetzt zwischen Monatsraster und Terminliste. Das war vorher an die Kachelgröße gekoppelt und ist in Wahrheit eine Frage dessen, was man sehen will. In der Menüleiste wird die Breite gemessen statt geschätzt. Vorher stand je Darstellungsart eine feste Zahl im Code, großzügig gewählt — mit dem Ergebnis, dass neben jedem Wert Platz für ein weiteres Symbol blieb, in dem nichts stand. Gemessen wird die breiteste vorkommende Zeichenfolge, nicht die gerade angezeigte, sonst springt die Leiste bei jedem Messwert. Die Balkendarstellung bekommt ihre Breite aus der Kernzahl, die erst nach der ersten Messung feststeht. Popover schließen jetzt beim Klick daneben. `behavior = .transient` genügt nicht: es greift, solange die eigene App aktiv ist, aber Onyx läuft als .accessory und wird durch einen Klick auf ein Statuselement nicht aktiviert. Ein Klick in ein fremdes Fenster erreichte das Popover gar nicht. Ein globaler Beobachter sieht solche Klicks, ein lokaler die in der eigenen App, Escape schließt ebenfalls — und es ist immer nur eines offen. Mixer und Lüfter haben eigene Menüleistenelemente. Beides regelt man mitten in etwas anderem; der Weg über ein Einstellungsfenster war zu weit. Die Lüfter sind damit auch als Panel-Kachel echt statt Platzhalter. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Scarriffle
@@ -42,6 +42,7 @@ public final class MenuBarController {
|
||||
}
|
||||
|
||||
public func stop() {
|
||||
closeOpenPopover()
|
||||
visibilityTimer?.invalidate()
|
||||
visibilityTimer = nil
|
||||
for id in entries.keys { setEnabled(false, forModuleID: id) }
|
||||
@@ -78,6 +79,12 @@ public final class MenuBarController {
|
||||
apply(entry.settings, toModuleID: id)
|
||||
}
|
||||
|
||||
/// Welches Popover gerade offen ist. Höchstens eines.
|
||||
private var openPopoverID: String?
|
||||
private var globalMonitor: Any?
|
||||
private var localMonitor: Any?
|
||||
private var keyMonitor: Any?
|
||||
|
||||
public func settings(forModuleID id: String) -> MenuBarModuleSettings? {
|
||||
entries[id]?.settings
|
||||
}
|
||||
@@ -100,7 +107,14 @@ public final class MenuBarController {
|
||||
button.sendAction(on: [.leftMouseUp, .rightMouseUp])
|
||||
button.identifier = NSUserInterfaceItemIdentifier(id)
|
||||
}
|
||||
entry.item.length = view.fittingSize.width > 0 ? view.fittingSize.width : NSStatusItem.variableLength
|
||||
applyWidth(view.intrinsicContentSize.width, to: entry)
|
||||
// Die nötige Breite steht nicht von Anfang an fest: die Kernzahl
|
||||
// kennt das Modul erst nach der ersten Messung, und ein Moduswechsel
|
||||
// ändert „100 %" in „100 °C".
|
||||
(view as? WidthReporting)?.onWidthChange = { [weak self] width in
|
||||
guard let self, let entry = entries[id] else { return }
|
||||
applyWidth(width, to: entry)
|
||||
}
|
||||
entry.module.activate()
|
||||
} else {
|
||||
entry.module.deactivate()
|
||||
@@ -108,20 +122,78 @@ public final class MenuBarController {
|
||||
checkVisibility()
|
||||
}
|
||||
|
||||
private func applyWidth(_ width: CGFloat, to entry: Entry) {
|
||||
entry.item.length = width > 0 ? width : NSStatusItem.variableLength
|
||||
}
|
||||
|
||||
// MARK: - Interaktion
|
||||
|
||||
@objc private func statusItemClicked(_ sender: NSStatusBarButton) {
|
||||
guard let id = sender.identifier?.rawValue, let entry = entries[id] else { return }
|
||||
|
||||
if entry.popover.isShown {
|
||||
entry.popover.performClose(nil)
|
||||
// Immer nur eines offen. Zwei Popover nebeneinander sehen aus wie ein
|
||||
// Fehler, und das zweite verdeckt oft das erste.
|
||||
if openPopoverID == id {
|
||||
closeOpenPopover()
|
||||
return
|
||||
}
|
||||
closeOpenPopover()
|
||||
|
||||
entry.popover.contentViewController = NSHostingController(
|
||||
rootView: entry.module.makePopoverView())
|
||||
entry.popover.show(relativeTo: sender.bounds, of: sender, preferredEdge: .minY)
|
||||
// Ohne das bleibt das Popover hinter dem aktiven Fenster.
|
||||
entry.popover.contentViewController?.view.window?.makeKey()
|
||||
openPopoverID = id
|
||||
startWatchingForOutsideClicks()
|
||||
}
|
||||
|
||||
// MARK: - Schließen bei Klick daneben
|
||||
|
||||
/// `NSPopover.behavior = .transient` genügt hier **nicht**.
|
||||
///
|
||||
/// Es schließt zuverlässig, solange die eigene App aktiv ist. Onyx läuft
|
||||
/// aber als `.accessory` und wird durch einen Klick auf ein Statuselement
|
||||
/// nicht aktiviert — ein Klick in ein fremdes Fenster erreicht das Popover
|
||||
/// dann gar nicht, und es bleibt offen stehen. Ein globaler Beobachter sieht
|
||||
/// solche Klicks trotzdem.
|
||||
private func startWatchingForOutsideClicks() {
|
||||
guard globalMonitor == nil else { return }
|
||||
globalMonitor = NSEvent.addGlobalMonitorForEvents(
|
||||
matching: [.leftMouseDown, .rightMouseDown, .otherMouseDown]) { [weak self] _ in
|
||||
Task { @MainActor in self?.closeOpenPopover() }
|
||||
}
|
||||
// Der globale Beobachter sieht keine Klicks in die eigene App — dafür
|
||||
// der lokale. Er reicht das Ereignis weiter, damit Knöpfe im Popover
|
||||
// weiterhin funktionieren.
|
||||
localMonitor = NSEvent.addLocalMonitorForEvents(
|
||||
matching: [.leftMouseDown, .rightMouseDown]) { [weak self] event in
|
||||
guard let self, let id = openPopoverID, let entry = entries[id] else { return event }
|
||||
// Klicks **im** Popover schließen es nicht.
|
||||
if event.window !== entry.popover.contentViewController?.view.window,
|
||||
event.window !== entry.item.button?.window {
|
||||
closeOpenPopover()
|
||||
}
|
||||
return event
|
||||
}
|
||||
// Auch die Escape-Taste und ein Bildschirmwechsel beenden es.
|
||||
keyMonitor = NSEvent.addLocalMonitorForEvents(matching: .keyDown) { [weak self] event in
|
||||
if event.keyCode == 53 { self?.closeOpenPopover(); return nil }
|
||||
return event
|
||||
}
|
||||
}
|
||||
|
||||
private func closeOpenPopover() {
|
||||
if let id = openPopoverID, let entry = entries[id] {
|
||||
entry.popover.performClose(nil)
|
||||
}
|
||||
openPopoverID = nil
|
||||
globalMonitor.map(NSEvent.removeMonitor)
|
||||
localMonitor.map(NSEvent.removeMonitor)
|
||||
keyMonitor.map(NSEvent.removeMonitor)
|
||||
globalMonitor = nil
|
||||
localMonitor = nil
|
||||
keyMonitor = nil
|
||||
}
|
||||
|
||||
private func checkVisibility() {
|
||||
|
||||
67
Packages/OnyxKit/Sources/OnyxMenuBar/MenuBarText.swift
Normal file
67
Packages/OnyxKit/Sources/OnyxMenuBar/MenuBarText.swift
Normal file
@@ -0,0 +1,67 @@
|
||||
import AppKit
|
||||
|
||||
/// Beschriftung und Breite eines Menüleisten-Elements.
|
||||
///
|
||||
/// Die Breite wird **gemessen**, nicht geschätzt. Vorher stand je Darstellungsart
|
||||
/// eine feste Zahl im Code, großzügig gewählt, damit nichts abgeschnitten wird —
|
||||
/// mit dem Ergebnis, dass neben jedem Wert Platz für ein weiteres Symbol blieb,
|
||||
/// in dem nichts stand. In einer Menüleiste ist das der teuerste Platz des
|
||||
/// ganzen Bildschirms.
|
||||
///
|
||||
/// Gemessen wird die **breiteste Zeichenfolge, die vorkommen kann**, nicht die
|
||||
/// gerade angezeigte. Sonst ändert das Element bei jeder Messung seine Größe und
|
||||
/// schiebt alle Symbole rechts davon hin und her.
|
||||
public enum MenuBarText {
|
||||
|
||||
/// Feste Ziffernbreite: ohne sie wackelt der Text bei jeder Messung.
|
||||
///
|
||||
/// Als berechnete Eigenschaft, nicht als gespeicherte: `NSFont` ist nicht
|
||||
/// `Sendable`, und die Schrift jedes Mal neu zu erfragen kostet nichts —
|
||||
/// AppKit gibt dieselbe zwischengespeicherte Instanz zurück.
|
||||
public static var font: NSFont {
|
||||
NSFont.monospacedDigitSystemFont(ofSize: 11, weight: .regular)
|
||||
}
|
||||
|
||||
public static let sidePadding: CGFloat = 2
|
||||
public static let symbolWidth: CGFloat = 13
|
||||
/// Abstand zwischen Symbol und Zahl.
|
||||
public static let innerGap: CGFloat = 2
|
||||
/// Schmaler geht nicht, ohne dass ein Glyph anstößt.
|
||||
public static let minimumWidth: CGFloat = 20
|
||||
|
||||
/// Die Breite eines Elements aus gemessener Textbreite. Rein, damit die
|
||||
/// Rechnung prüfbar bleibt.
|
||||
public static func itemWidth(textWidth: CGFloat, includesSymbol: Bool) -> CGFloat {
|
||||
let symbol = includesSymbol ? symbolWidth + innerGap : 0
|
||||
let content = textWidth > 0 ? ceil(textWidth) + 2 * sidePadding : 2 * sidePadding
|
||||
return max(minimumWidth, content + symbol)
|
||||
}
|
||||
|
||||
public static func width(of text: String) -> CGFloat {
|
||||
guard !text.isEmpty else { return 0 }
|
||||
return NSAttributedString(string: text, attributes: [.font: font]).size().width
|
||||
}
|
||||
|
||||
/// Die Breite für die breiteste vorkommende Zeichenfolge.
|
||||
public static func width(for reference: String, includesSymbol: Bool = false) -> CGFloat {
|
||||
itemWidth(textWidth: width(of: reference), includesSymbol: includesSymbol)
|
||||
}
|
||||
|
||||
public static func draw(_ text: String, color: NSColor, in rect: NSRect) {
|
||||
let string = NSAttributedString(string: text,
|
||||
attributes: [.font: font, .foregroundColor: color])
|
||||
let size = string.size()
|
||||
string.draw(at: NSPoint(x: rect.midX - size.width / 2,
|
||||
y: rect.midY - size.height / 2))
|
||||
}
|
||||
}
|
||||
|
||||
/// Eine Statusansicht, deren nötige Breite sich ändern kann.
|
||||
///
|
||||
/// Der Controller stellt das Statuselement daraufhin neu ein. Ohne diesen Weg
|
||||
/// bliebe ein Element in der Breite stehen, die beim allerersten Zeichnen
|
||||
/// gebraucht wurde — vor der ersten Messung also.
|
||||
@MainActor
|
||||
public protocol WidthReporting: AnyObject {
|
||||
var onWidthChange: ((CGFloat) -> Void)? { get set }
|
||||
}
|
||||
Reference in New Issue
Block a user