Skip to content

Commit 7ba5d49

Browse files
committed
Hide trailing indicators in catalog models and expanded family headers
Removed visual clutter by hiding trailing indicators when they don't provide actionable information: - Catalog models: hide arrow.down indicator for compatible models (keep nosign for incompatible) - Family headers: hide chevron when expanded (keep chevron.right when collapsed with size labels) This creates a cleaner, more focused UI where indicators only appear when they convey meaningful state information.
1 parent a0ac990 commit 7ba5d49

File tree

2 files changed

+15
-8
lines changed

2 files changed

+15
-8
lines changed

LlamaBarn/Menu/CatalogModelItemView.swift

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -161,16 +161,18 @@ final class CatalogModelItemView: ItemView {
161161

162162
// Status-specific display
163163
// Catalog items only show available models (compatible or incompatible)
164-
let symbolName = compatible ? "arrow.down" : "nosign"
165-
statusIndicator.image = NSImage(systemSymbolName: symbolName, accessibilityDescription: nil)
164+
// Show nosign indicator for incompatible models, hide for compatible
165+
if compatible {
166+
statusIndicator.isHidden = true
167+
} else {
168+
statusIndicator.isHidden = false
169+
statusIndicator.image = NSImage(systemSymbolName: "nosign", accessibilityDescription: nil)
170+
statusIndicator.contentTintColor = Typography.tertiaryColor
171+
}
166172

167173
// No tooltips for catalog items
168174
toolTip = nil
169175

170-
// Colors: status indicator only (label color is handled in makeModelNameAttributedString)
171-
statusIndicator.contentTintColor =
172-
compatible ? Typography.secondaryColor : Typography.tertiaryColor
173-
174176
// Clear highlight if no longer actionable
175177
if !highlightEnabled { setHighlight(false) }
176178
needsDisplay = true

LlamaBarn/Menu/FamilyHeaderView.swift

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,13 @@ final class FamilyHeaderView: ItemView {
7575
}
7676

7777
private func updateChevron() {
78-
let imageName = isCollapsed ? "chevron.right" : "chevron.down"
79-
chevronImageView.image = NSImage(systemSymbolName: imageName, accessibilityDescription: nil)
78+
if isCollapsed {
79+
chevronImageView.isHidden = false
80+
chevronImageView.image = NSImage(
81+
systemSymbolName: "chevron.right", accessibilityDescription: nil)
82+
} else {
83+
chevronImageView.isHidden = true
84+
}
8085
}
8186

8287
private func updateAccessibilityLabel() {

0 commit comments

Comments
 (0)