Applied-Software/ghostty
5
0
Fork 0
mirror of https://github.com/ghostty-org/ghostty.git synced 2025-08-02 14:57:31 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fixing deprecation warnings, experiencing new errors.

Browse Source
This commit is contained in:
johnseth97
2024-10-19 18:53:29 -04:00
parent 0084103ff1
commit 96a0ca613a
1 changed files with 26 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
macos/Sources/Features/Terminal/TerminalToolbar.swift
View File

@ -43,17 +43,18 @@ class TerminalToolbar: NSToolbar, NSToolbarDelegate {
item.view = self.titleTextField item.view = self.titleTextField
item.visibilityPriority = .user item.visibilityPriority = .user
// NSToolbarItem.minSize and NSToolbarItem.maxSize are deprecated, and make big ugly self.titleTextField.translatesAutoresizingMaskIntoConstraints = false
// warnings in Xcode when you use them, but I cannot for the life of me figure out
// how to get this to work with constraints. The behavior isn't the same, instead of
// shrinking the item and clipping the subview, it hides the item as soon as the
// intrinsic size of the subview gets too big for the toolbar width, regardless of
// whether I have constraints set on its width, height, or both :/
//
// If someone can fix this so we don't have to use deprecated properties: Please do.
item.minSize = NSSize(width: 32, height: 1)
item.maxSize = NSSize(width: 1024, height: self.titleTextField.intrinsicContentSize.height)
self.titleTextField.setContentHuggingPriority(.defaultLow, for: .horizontal)
self.titleTextField.setContentCompressionResistancePriority(.defaultHigh, for: .horizontal)
// Add constraints to the toolbar item's view
NSLayoutConstraint.activate([
// Set the height constraint to match the toolbar's height
self.titleTextField.heightAnchor.constraint(equalToConstant: 22), // Adjust as needed
])
item.isEnabled = true item.isEnabled = true
case .resetZoom: case .resetZoom:
item = NSToolbarItem(itemIdentifier: .resetZoom) item = NSToolbarItem(itemIdentifier: .resetZoom)
@ -73,23 +74,28 @@ class TerminalToolbar: NSToolbar, NSToolbarDelegate {
// getting smaller than the max size so starts clipping. Lucky for us, two of the // getting smaller than the max size so starts clipping. Lucky for us, two of the
// built-in spacers plus the un-zoom button item seems to exactly match the space // built-in spacers plus the un-zoom button item seems to exactly match the space
// on the left that's reserved for the window buttons. // on the left that's reserved for the window buttons.
return [.titleText, .flexibleSpace, .space, .space, .resetZoom] return [.flexibleSpace, .titleText, .flexibleSpace]
} }
} }
/// A label that expands to fit whatever text you put in it and horizontally centers itself in the current window. /// A label that expands to fit whatever text you put in it and horizontally centers itself in the current window.
fileprivate class CenteredDynamicLabel: NSTextField { fileprivate class CenteredDynamicLabel: NSTextField {
override func viewDidMoveToSuperview() { override func viewDidMoveToSuperview() {
// Truncate the title when it gets too long, cutting it off with an ellipsis. // Configure the text field
isEditable = false
isBordered = false
drawsBackground = false
alignment = .center
lineBreakMode = .byTruncatingTail
cell?.truncatesLastVisibleLine = true cell?.truncatesLastVisibleLine = true
cell?.lineBreakMode = .byCharWrapping
// Use Auto Layout
// Make the text field as small as possible while fitting its text. translatesAutoresizingMaskIntoConstraints = false
setContentHuggingPriority(.required, for: .horizontal)
cell?.alignment = .center // Set content hugging and compression resistance priorities
setContentHuggingPriority(.defaultLow, for: .horizontal)
// We've changed some alignment settings, make sure the layout is updated immediately. setContentCompressionResistancePriority(.defaultHigh, for: .horizontal)
needsLayout = true
} }
} }