microsoft / microsoft/fluentui-react-native
Components: Generate TeachingPopover
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 1.4k
- Forks
- 179
- Avg merge
- 16h 17m
- Merged PRs (30d)
- 30
Description
Summary
Generate TeachingPopover with its canonical contract, implementation, tests, stories, platform validation, and explicit package export so contextual product education is available. The local spec must be ingested, and the component remains downstream of the shared Popover positioning, dismissal, and focus behavior.
Observed. teaching-popover is entry teaching-popover in the canonical catalog and has no
implementation in packages/agentic/components. Upstream describes it as: Contextual coachmark that teaches existing users about a new or recently changed feature. Composes a Popover surface with an opinionated content layout: optional header with a close control, optional top-aligned image, required title, body, and a footer action row (Secondary, Stepper, Primary). Covers Step mode (Single/Multi), Header, and Image.
Goal
Generate the teaching-popover component in
packages/agentic/components/src/components against the
canonical Fluent Flex spec, with the colocated coverage, stories, and platform
validation the package requires, and export it explicitly from
src/index.ts.
Stage
Stage 2 (beta). Wave 2 - overlay and popover stack.
Why it matters
Observed. The canonical catalog has 50 components and the agentic library has
generated 18. teaching-popover is one of the 32 in the gap, enumerated in
the missing-components index.
Inferred. It belongs to the overlay stack, which is the largest single cluster of blocked components. Delivering the cluster converts a broad "not implementable" state into ordinary component work.
Observed current state
Canonical spec and source
| Field | Value |
|---|---|
| Canonical spec | plugins/components/skills/teaching-popover/SKILL.md with usage.md, web/tokens.yaml, web/accessibility.md, and web/interaction.md, at x3-design/fluent-design @ d334acf |
| Upstream tier | molecule |
| Upstream catalog key | teaching-popover in component-map.yaml |
| Local staged spec | None. No specs/teaching-popover folder exists; the spec must be ingested first |
| Local implementation | None. No src/components/teaching-popover directory exists |
Closest existing FURN or V1 implementation
- No coachmark exists in any FURN catalog. The nearest infrastructure is
packages/native/Callout.
Note on platform canon. Where a legacy package has platform-specific files, the
desktop path (.win32, .windows, .macos, or the unsuffixed cross-platform file)
is the compatibility reference. .ios, .android, and .mobile variants are not
treated as the cross-platform canonical implementation.
Local dependencies
src/components/button- Observed present (component).
Known blockers
Observed. There is no recorded blocker, because there is no local spec folder and therefore no
blockers.md. The absence of an ingested spec is itself the first blocker; see
spec-ingestion-and-generation.md.
Scope
- Ingest or promote the spec into the component directory in the generated shape
(SPEC.mdplus aspec/folder), per
spec-ingestion-and-generation.md. - Establish the public contract first: variants, slots, native props, accessibility,
interaction states, and platform behavior. - Implement the required file structure from
src/components/AGENTS.md:
teaching-popover.types.ts,teaching-popover.styles.ts,use*.ts,use*Styles.ts,render*.tsx, and
the assembly module with adisplayName. - Style from Flex tokens first and record genuine token gaps rather than substituting
unrelated values. - Export the component, its public types, and the state, style, and render stages
explicitly fromsrc/index.tsunder component-qualified unstable names. - Consume the overlay host from
popover; this component owns only the content layout and the step model. - Decide whether the multi-step stepper owns its index or the caller does, and classify the axis accordingly.
Out of scope
- Changing the contract of an existing generated component, unless this task explicitly
calls for it above; raise it as its own item. - Building shared infrastructure owned by another task: overlay hosting
(popover), roving focus (focus-zone.md), spec
ingestion (spec-ingestion-and-generation.md),
or the coverage contract (component-test-strategy.md). - Migrating any legacy FURN consumer onto this component.
- Mobile platform support; the applicable upstream companion is
web, retargeted to
React Native for Windows and macOS.
Deliverables
src/components/teaching-popover/implementing the full required file structure.SPEC.mdand aspec/companion folder with provenance recording the upstream path,
SHA, and ingestion date.- Colocated
teaching-popover.test.tsx,teaching-popover.types.test.ts, andteaching-popover.stories.tsx. - Snapshot coverage where the visual contract is state-dependent, or a recorded
exemption. - Explicit exports from
src/index.ts, withsrc/index.test.tsupdated. - A changeset, per the repository's version management rules.
- A recorded readiness level with its supporting evidence.
Required coverage, stories, and platform validation
Runtime and type coverage:
- Both Step modes, header present and absent, image present and absent.
- The footer action row layout in each combination.
- Stepper advances and retreats, and the boundary steps disable the correct controls.
- Close control accessible name; the image is decorative unless labeled.
- Title is required at the type level.
- User style and handler forwarding, and that user styles are applied after component
styles. - Compile-time slot acceptance and any mutually exclusive props.
Stories: a typed Meta titled Components/TeachingPopover, an args-driven Default, a
grouped Overview when the contract has several axes, one focused story per variant
axis, and an interactive story for each stateful axis. Self-driving axes expose
default<State> as the control; externally driven selection is owned by the story with
React.useState. Identity-changing axes are not exposed as controls.
Platform validation: the component's stories render and behave correctly on macOS
and on React Native Windows Fabric through apps/storybook,
including hover, pressed, disabled, focus, optional-slot, and constrained-content
scenarios. Add a stable root testID if the component joins the automated smoke set.
Acceptance criteria
-
src/components/teaching-popoverexists with the required file structure and adisplayName. - Every variant axis in the component's
SPEC.mdis implemented, or a deviation is recorded in the spec. - The component styles from Flex tokens, with any genuine token gap recorded rather than approximated.
- Focus feedback renders through
FocusVisual; nooutline*props and no React Native Windows native focus visuals are introduced. - State precedence is explicit: disabled wins over pressed, which wins over hovered.
- Every stateful axis is classified as self-driving or externally driven, and both paths are covered.
- Runtime, type, and story coverage satisfy the package coverage contract.
- The component, its public types, and its pipeline stages are exported explicitly from
src/index.ts, andsrc/index.test.tscovers them. - The component's stories render on macOS and on React Native Windows Fabric.
-
yarn workspace @fluentui-react-native/components format,lint,build, andtestpass, and the Storybook bundle succeeds for both desktop platforms. - A readiness level is recorded with its supporting evidence.
- A changeset is included.
Dependencies and ordering
- Blocked by spec-ingestion-and-generation.md: the spec must be pulled from the pinned upstream revision before generation can start.
- Blocked by
popoverfor the overlay host, anchoring, light dismiss, and focus return. - Gated by component-test-strategy.md: the component must satisfy the coverage contract on arrival rather than being retrofitted.
- Gated by readiness-model.md: the target readiness level must be stated before the work starts.
- Consumes the Infrastructure workstream (
storybook-e2e.md) for automated macOS and Windows Fabric validation; until it lands, platform validation is manual.
Risks and open decisions
- It is a thin layer over
popover, so it is a good early proof that the overlay host is reusable - but it is worthless before that host exists. - Image sizing inside an anchored surface interacts with the edge-flip behavior and needs on-device checking.
Evidence and references
- Upstream spec:
plugins/components/skills/teaching-popover/SKILL.mdand
components-index/SKILL.md
atx3-design/fluent-design@d334acf. - Canonical implementations to follow:
src/components/button
for a styled higher-order component andsrc/primitives/icon
for a primitive. - Authoring rules:
src/components/AGENTS.mdand the
agentic component authoring skill. - Workstream context: Components README and
the missing-components index.
Sources retrieved 2026-08-20.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with spec-ingestion-and-generation.md and the pinned upstream teaching-popover spec, then read src/components/AGENTS.md and the existing button and popover work. Establish the contract and dependency readiness before creating src/components/teaching-popover. Done means the required implementation, colocated tests and stories, explicit exports, platform validation, changeset, and passing workspace and Storybook checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native, typescript
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100