Skip to content

docs(spec): Tooltip records whether its press toggle stays implicit as an open question - #9712

Closed
vjeux wants to merge 1 commit into
meta:mainfrom
vjeux:docs/tooltip-implicit-toggle-oq
Closed

vjeux wants to merge 1 commit into
meta:mainfrom
vjeux:docs/tooltip-implicit-toggle-oq

Conversation

@vjeux

@vjeux vjeux commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Exact decision needed

None yet: an open API question on component:Tooltip, whether triggerBehavior="toggle" keeps taking over its child button's press implicitly, or moves to an explicit pattern where the caller's button calls show() and hide().

Existing authority searched

component:Tooltip (FR9–FR10, DEC log, Open questions: "None."), component:InfoTip, and useTooltip's TooltipReturn; useHoverCard already returns show() and hide(), a precedent for the explicit option. Open PRs that edit Tooltip.spec.md (#9291, #9361) leave its Open questions section alone. No current claim decides between the two patterns.

Decision

No decision: OQ1 records the question (human-api, low priority), with no code change.

Owner's text, adjusted for accuracy at main

The owner's text was: "OQ — Implicit toggle. With triggerBehavior="toggle", Tooltip takes over its child button's press implicitly. Open: whether toggle stays implicit or moves to the explicit pattern InfoTip uses, where the caller's Button calls useTooltip's show() and hide()."

Two facts differ at main 4a8a76e, so the record words them as possibilities:

  • InfoTip composes <Tooltip triggerBehavior="toggle">, the implicit path itself.
  • useTooltip returns ref, positionRef, interactionRef, anchorId, describedBy and renderTooltip, with no show() or hide().

Rejected alternatives

None; the question stays open.

Claim scope and non-goals

Only Open questions changes; no behavior, API or verification claim moves.

Evidence and implementation relationship

None; no implementation PR.

Writing rubric results

  • R1 narration: none; OQ1 states the system in the present tense.
  • R2 research in the record: none.
  • R3 evidence naming a change: none.
  • R4 rejected alternatives over one line or outside a decision: none.
  • R5 pull request or issue references remaining: none.
  • R6 ruled open questions still listing options: none; OQ1 is unanswered and keeps its two options.

Scope

  • This PR contains specification records only.
  • It records observable behavior and preserves equivalent internal implementations.
  • Any named internal mechanism is intentionally public, names its dependent caller/system, and explains why an equivalent implementation would not satisfy the contract.
  • Verification states the evidence layer and failure signal without prescribing CI job/workflow topology.
  • It records one intentional durable decision, not a rescue for a separable tagalong.
  • The canonical knowledge owner, affected current claims, and overlapping open work are named; any ownership collision is resolved or called out for human decision.
  • Adjacent API, visual, accessibility, composition, and implementation facts remain outside scope unless this exact decision depends on them.
  • Public text and artifacts contain no internal Meta context.

Validation

At 3ccce21: pnpm check:knowledge, Prettier and git diff --check pass. The owner-gated record needs an engineering owner's /approve-spec.

…s an open question

`triggerBehavior="toggle"` takes over the child button's press
implicitly. Whether it stays that way or moves to an explicit pattern,
with the caller's button calling show() and hide(), is an open API
question. This records it, with no decision and no code change.
@vercel

vercel Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
astryx Ready Ready Preview Nov 3, 2026 5:23am UTC

Request Review

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Nov 3, 2026
@astracat-bot

astracat-bot Bot commented Nov 3, 2026

Copy link
Copy Markdown

AI review status for this pull request.

Review status Updated
🔄 In progress (for maintainers only) Nov 3, 2026, 5:13 AM UTC

@github-actions github-actions Bot added the needs:spec-owner-review Current knowledge records await owner approval label Nov 3, 2026
@vjeux

vjeux commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Closing: covered by #9711's Tooltip open question.

@vjeux vjeux closed this Nov 3, 2026
@vjeux
vjeux deleted the docs/tooltip-implicit-toggle-oq branch November 3, 2026 05:16

This branch was successfully deployed

1 active deployment
Preview — 3ccce219 Deployed Nov 3, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. needs:spec-owner-review Current knowledge records await owner approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant