@godxjp/ui 18.12.18 → 18.12.19
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -141,8 +141,15 @@ const AppSettingPicker = React.forwardRef(
|
|
|
141
141
|
className: cn(
|
|
142
142
|
inline ? "ui-app-setting-picker-inline h-auto min-h-0 w-auto rounded-none border-0 bg-transparent p-0 shadow-none" : iconOnly ? (
|
|
143
143
|
// Structurally icon-only: drop the owned width + value spacing and square the box to
|
|
144
|
-
// the density-aware --control-height tap target, centring the icon.
|
|
145
|
-
|
|
144
|
+
// the density-aware --control-height tap target, centring the icon. Visually it
|
|
145
|
+
// sits among ghost icon buttons (the topbar's sidebar-toggle/notifications/account
|
|
146
|
+
// triggers), so it drops controlTriggerClass's form-input chrome (border/bg/shadow)
|
|
147
|
+
// at rest and adopts the same ghost hover — a resting border here read as visually
|
|
148
|
+
// inconsistent next to its borderless topbar siblings. The open-state ring
|
|
149
|
+
// (`data-[state=open]:border-ring`, from controlTriggerClass) and the
|
|
150
|
+
// focus-visible ring are untouched, so keyboard and "is this open" affordance
|
|
151
|
+
// still hold.
|
|
152
|
+
"ui-app-setting-picker-icon w-[length:var(--control-height)] justify-center border-transparent bg-transparent ps-0 pe-0 shadow-none hover:bg-accent hover:text-accent-foreground"
|
|
146
153
|
) : (
|
|
147
154
|
// Labeled: sized to a per-kind width from `sm` up; below `sm` it hugs its content and
|
|
148
155
|
// caps at the container (`w-auto max-w-full`) instead of the old UNCONDITIONAL
|