@flamingo-stack/openframe-frontend-core 0.0.220-snapshot.20260602172647 → 0.0.220

package/src/components/layout/page-heading.tsx

@@ -2,15 +2,14 @@ import type { ReactNode } from 'react'
 
 /**
  * THE page-title style — the ODS `text-h1` token (`--font-size-h1-title` =
- * 40 / 48 / 56px, Azeret Mono semibold), which already owns the responsive
- * letter-spacing (-0.02em). Mirrors the global `h1 { @apply text-h1 }` base, so
- * every page heading is the SAME size across the lib and every consuming app.
+ * 40 / 48 / 56px, Azeret Mono semibold) plus the standard heading tracking.
+ * Mirrors the global `h1 { @apply text-h1 }` base, so every page heading is
+ * the SAME size across the lib and every consuming app.
  *
- * Single source of truth — do NOT hardcode a px size or re-assert the token's
- * tracking; render `<PageHeading>` (or, for the rare non-h1 caller, apply
- * `PAGE_HEADING_CLASS`).
+ * Single source of truth — do NOT hardcode a px size for a page title; render
+ * `<PageHeading>` (or, for the rare non-h1 caller, apply `PAGE_HEADING_CLASS`).
  */
-export const PAGE_HEADING_CLASS = 'text-h1 text-ods-text-primary'
+export const PAGE_HEADING_CLASS = 'text-h1 text-ods-text-primary tracking-[-1.12px]'
 
 const DESCRIPTION_CLASS =
   "mt-6 max-w-[640px] font-['DM_Sans'] text-[16px] md:text-[18px] leading-[24px] md:leading-[28px] text-ods-text-secondary"
@@ -46,15 +45,10 @@ export function PageHeading({
 }: PageHeadingProps) {
   const headingClass = className ? `${PAGE_HEADING_CLASS} ${className}` : PAGE_HEADING_CLASS
   const descClass = descriptionClassName ? `${DESCRIPTION_CLASS} ${descriptionClassName}` : DESCRIPTION_CLASS
-  // `description` is a ReactNode, so `description={cond && '...'}` can pass a
-  // boolean `false` — exclude it (and empty string) so we never render an empty
-  // <p> that adds phantom vertical gap beneath the heading.
-  const hasDescription =
-    description != null && description !== '' && typeof description !== 'boolean'
   return (
     <>
       <Tag className={headingClass}>{children}</Tag>
-      {hasDescription && <p className={descClass}>{description}</p>}
+      {description != null && description !== '' && <p className={descClass}>{description}</p>}
     </>
   )
 }

package/src/stories/GuideWelcome.stories.tsx

@@ -14,18 +14,6 @@ const ACTIONS = [
   { id: 'find', label: 'Find', onClick: () => console.log('find') },
 ]
 
-// Quick actions are caller-provided (GuideWelcome ships no defaults); these
-// demonstrate the inline chips + the "⋯" overflow menu.
-const SAMPLE_QUICK_ACTIONS = [
-  { id: 'how-to-start', label: 'How to start' },
-  { id: 'connect-device', label: 'Connect device' },
-  { id: 'find-device', label: 'Find device' },
-  { id: 'remote-connection', label: 'Remote connection' },
-  { id: 'run-scripts', label: 'Run scripts' },
-  { id: 'device-software', label: 'Device software' },
-  { id: 'bulk-update', label: 'Bulk update' },
-]
-
 const SAMPLE_LIST = (
   <div className="shrink-0 overflow-hidden rounded-md border border-ods-border">
     <MingoOnboardingCard
@@ -91,7 +79,6 @@ type Story = StoryObj<typeof GuideWelcome>
 /** Full guide-mode empty state with the slash-command list and quick actions. */
 export const Default: Story = {
   args: {
-    quickActions: SAMPLE_QUICK_ACTIONS,
     onQuickAction: (a) => console.log('quick action', a.id),
     children: SAMPLE_LIST,
   },
@@ -100,24 +87,15 @@ export const Default: Story = {
 /** No slash commands available yet — greeting + quick actions only. */
 export const NoCommands: Story = {
   args: {
-    quickActions: SAMPLE_QUICK_ACTIONS,
     onQuickAction: (a) => console.log('quick action', a.id),
   },
 }
 
-/** No quick actions supplied (the default) — the chip row is omitted entirely. */
-export const NoQuickActions: Story = {
-  args: {
-    children: SAMPLE_LIST,
-  },
-}
-
 /** Custom greeting copy. */
 export const CustomCopy: Story = {
   args: {
     title: 'Ask the Guide',
     subtitle: 'A temporary session for exploring docs and tickets.',
-    quickActions: SAMPLE_QUICK_ACTIONS,
     onQuickAction: (a) => console.log('quick action', a.id),
     children: SAMPLE_LIST,
   },