-
[OPIK-6425] [FE] Address review: BI dedup, aria-busy, align popover shadow
发布于
2026-06-23 11:56:48 +00:00 - ExplainButton: fire EXPLAIN_CLICKED only on a fresh dispatch — reopening a
popover on a cached/in-flight cell reuses the entry and no longer recounts,
so the CLICKED→COMPLETED funnel isn't inflated. Locked in with a test. - ExplainButton: drop the hard-coded rgba PopoverContent shadow and inherit the
shared shadow-md, matching all 44 other PopoverContent call sites (the popover
was the only one overriding it; modal + onCloseAutoFocus already match
FilterChipPopover/PromptLibraryMenu). - ExplainPopover: add aria-busy on the live region so screen readers announce
the settled answer once instead of re-reading every streamed token; collapse
the duplicated Thinking/waking blocks into one with a derived label.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
下载附件
- ExplainButton: fire EXPLAIN_CLICKED only on a fresh dispatch — reopening a