Repository navigation
objectui: UserFilters 里三个 button 元素缺 type="button"(objectui#3344 家族,当前休眠)—— observation #6952
Description
Activity
Triage (maintainer-authorized one-off pass, 2026-08-09, registered on #6015): promoted
finding→pm:queue(routingrepo:objectuikept). Dormant today, but the fix is threetype="button"attributes with an in-family precedent (objectui#3344) — holding costs more than fixing, and the failure mode when composition changes (a filter click submitting the enclosing form) is the silent kind. The filing's harder second half (making the primitive default totype="button"in@object-ui/components) is explicitly NOT in scope of this card — that is a separate design card if ever wanted.
Generated by Claude Code
🔒 认领(objectui 分片 PM,session
session_01GTRjn8xBqp75dk7kFupVRt,恢复批次 1 补位)- 分支:
claude/issue-os6952-userfilters-button-type(objectui 仓) - 范围(正文三处):
packages/plugin-list/src/UserFilters.tsx的 chip/clear、user-filters-morePopoverTrigger、预设 tab 按钮补type="button";⛔「PopoverTrigger 原语自带默认 type」的结构性修法不做(动 components 全消费面,另裁)。行号以现况为准(objectui:allowAddTab渲染出的「新增 tab」按钮没有 click handler —— 契约刚声明的能力只交付了一半 #5236 的 PR3926 已改此文件)。 - 文件面:
packages/plugin-list/src/UserFilters.tsx+ 就近测试。
Generated by Claude Code
- 分支:
发现于 objectstack#5236 的实施(给
allowAddTab按钮接上会话级行为,objectui PR #3926)。不阻塞那一单,它已经给自己新加的 add 触发器显式写了type="button";这是顺手看到的既有面,单独立案。事实(实测,非阅读推断)
objectui/packages/plugin-list/src/UserFilters.tsx里的 button 元素,按git grep逐个数(行号取自 objectuiorigin/main=5e524950d):type:511:672user-filters-more溢出 PopoverTrigger:855filter-tab-*)HTML 里 button 元素在 form 元素内部默认
type="submit",而 Radix 的PopoverTrigger不preventDefault—— 这正是 objectui#3344 的形态:combobox 触发器因此在对象表单里「点一下下拉就把表单提交了」,那一单的修法是给触发器显式声明type="button"(并留了type透传给显式覆盖的人)。上表三个是同一家族里没被那一单覆盖到的同类面。为什么标 observation、不进 pm:queue
今天用户碰不到:
UserFilters的实际挂载点是ListView的工具条(ListView.tsx的data-testid="user-filters"那一格),不在 form 元素里;要触发得有人把 ListView 组合进一个 form 里渲染。所以这是休眠的潜在缺陷,不是在线 bug —— 按 objectstack#4949 的分诊纪律标finding,严重度交给 PM 的 triage 轮判,我不自己定级。修的话是什么形状
三处各加
type="button",与 objectui#3344 同款;若要一次性防回归,更结实的做法是在@object-ui/components侧让PopoverTrigger这类原语自带默认type(#3344 已经在 combobox 上开了这个先例),否则每个新写的触发器都得靠作者记得 —— 那是「靠人不犯错」而不是「结构上不可能犯错」。两种形状的取舍归维护者。参考
type="button",含packages/components/src/__tests__/combobox-trigger-type.test.tsx那三条钉子)filter-tab-add触发器已带type="button",注释里引了 feat(approvals): server-computed viewer capability for precise decision-action gating (#3310) #3344)🤖 Generated with Claude Code
https://claude.ai/code/session_01GTRjn8xBqp75dk7kFupVRt