@skyhook-io/k8s-ui 1.14.1 → 1.14.3

package/src/components/ui/Collapse.tsx

@@ -37,7 +37,10 @@ export function Collapse({
   const render = !mountLazily || hasOpened
   return (
     <div
-      className={clsx('grid transition-[grid-template-rows] duration-200 ease-out', className)}
+      className={clsx(
+        'grid transition-[grid-template-rows] duration-200 ease-out motion-reduce:transition-none',
+        className,
+      )}
       style={{ gridTemplateRows: open ? '1fr' : '0fr' }}
     >
       {/* `relative` is load-bearing. Collapsed content is still laid out at full
@@ -61,7 +64,11 @@ export function CollapseChevron({ open, className }: { open: boolean; className?
   return (
     <ChevronRight
       aria-hidden="true"
-      className={clsx('shrink-0 text-theme-text-tertiary transition-transform duration-200', open && 'rotate-90', className)}
+      className={clsx(
+        'shrink-0 text-theme-text-tertiary transition-transform duration-200 motion-reduce:transition-none',
+        open && 'rotate-90',
+        className,
+      )}
     />
   )
 }

package/src/components/ui/ConfirmDialog.test.tsx

@@ -0,0 +1,37 @@
+import { type ReactNode } from 'react'
+import { renderToStaticMarkup } from 'react-dom/server'
+import { describe, expect, it, vi } from 'vitest'
+import { ConfirmDialog } from './ConfirmDialog'
+
+vi.mock('./DialogPortal', () => ({
+  DialogPortal: ({ children }: { children: ReactNode }) => <>{children}</>,
+}))
+
+const props = {
+  open: true,
+  onClose: () => {},
+  onConfirm: () => {},
+  title: 'Share this investigation?',
+  message: 'Organization members can read and continue this investigation.',
+  confirmLabel: 'Share with organization',
+}
+
+describe('ConfirmDialog warning disclosure', () => {
+  it('can omit redundant boilerplate without removing the actual disclosure or actions', () => {
+    const html = renderToStaticMarkup(<ConfirmDialog {...props} variant="warning" showWarning={false} />)
+    expect(html).toContain(props.message)
+    expect(html).toContain(props.confirmLabel)
+    expect(html).toContain('Cancel')
+    expect(html).not.toContain('Please confirm you want to proceed')
+  })
+
+  it('preserves warning boilerplate by default for existing callers', () => {
+    expect(renderToStaticMarkup(<ConfirmDialog {...props} variant="warning" />))
+      .toContain('Please confirm you want to proceed with this action.')
+  })
+
+  it('preserves the irreversible-action warning for destructive callers', () => {
+    expect(renderToStaticMarkup(<ConfirmDialog {...props} />))
+      .toContain('This action cannot be undone.')
+  })
+})

package/src/components/ui/ConfirmDialog.tsx

@@ -18,6 +18,7 @@ interface ConfirmDialogProps {
   isLoading?: boolean
   isClosable?: boolean // Allow closing even when isLoading (e.g., for long-running ops the user can dismiss)
   confirmDisabled?: boolean // Block the confirm action while custom content is invalid (e.g., bad YAML)
+  showWarning?: boolean
   className?: string
   children?: ReactNode // Optional custom content (e.g., checkboxes)
 }
@@ -35,6 +36,7 @@ export function ConfirmDialog({
   isLoading = false,
   isClosable = false,
   confirmDisabled = false,
+  showWarning = true,
   className,
   children,
 }: ConfirmDialogProps) {
@@ -84,7 +86,7 @@ export function ConfirmDialog({
       )}
 
       {/* Warning message — hidden once the action is in progress */}
-      {!isLoading && !children && (
+      {showWarning && !isLoading && !children && (
         <div className="p-4">
           <div
             className={clsx(

package/src/components/ui/drawer-components.tsx

@@ -291,7 +291,7 @@ function isConditionHealthy(cond: { type?: string; status?: string }): boolean {
 
 export type ConditionTone = 'ok' | 'warning' | 'fail' | 'unknown'
 
-function defaultConditionTone(cond: { type?: string; status?: string }): ConditionTone {
+export function defaultConditionTone(cond: { type?: string; status?: string }): ConditionTone {
   if (cond.status !== 'True' && cond.status !== 'False') return 'unknown'
   return isConditionHealthy(cond) ? 'ok' : 'fail'
 }

package/src/components/workload/WorkloadView.tsx

@@ -728,10 +728,11 @@ export function WorkloadView({
   // prominent "Diagnose" on a problem, a quiet icon when fine. Rendered via the host
   // slot (DiagnoseCustomization); standalone Radar injects it, Hub overrides it.
   const renderDiagnose = actionsBarProps?.renderDiagnose as
-    | ((ctx: { kind: string; namespace: string; name: string; health?: DiagnoseHealthHint }) => ReactNode)
+    | ((ctx: { kind: string; group?: string; namespace: string; name: string; health?: DiagnoseHealthHint }) => ReactNode)
     | undefined
   const diagnoseAction = renderDiagnose?.({
     kind: apiKind,
+    group,
     namespace,
     name,
     health: diagnoseHealthHint(apiKind, resource),