Skip to content

fix(plugin-gantt): 删除冗余行「定位」图标;「→」详情按钮改独立操作槽#2487

Merged
baozhoutao merged 1 commit into
mainfrom
claude/gantt-row-open-ux-2482
Jul 14, 2026
Merged

fix(plugin-gantt): 删除冗余行「定位」图标;「→」详情按钮改独立操作槽#2487
baozhoutao merged 1 commit into
mainfrom
claude/gantt-row-open-ux-2482

Conversation

@baozhoutao

Copy link
Copy Markdown
Contributor

Closes #2482

问题

#2460 落地「行单击=Focus 定位」后,任务列表行右缘遗留两个体验问题:

  1. 「定位」十字准星悬停图标冗余:它是旧交互(行单击=开抽屉)时代的定位入口,现在整行单击即定位,图标失去价值还加剧右缘拥挤。
  2. 「→」详情按钮绝对定位叠在「结束」列上、热区过小:两个按钮 position: absolute 锚在行右缘,悬停显示时直接压在结束日期文字上;按钮无内边距,点击热区只有 12×12px 裸图标。

修复(GanttView.tsx)

  • 删除定位按钮及其 i18n 键 gantt.row.locate(useGanttTranslation 兜底 + zh/en locale);定位能力由行单击承担,无功能损失。
  • 「→」按钮改为行 flex 布局中的独立固定操作槽(24px 宽、常驻占位):
    • 与日期列零重叠,悬停不再遮字;
    • 按钮 24×24px 圆角热区 + hover:bg-accent 底色,图标 14px;
    • 保留悬停渐显(行悬停 60% → 按钮自悬停/键盘聚焦 100%)与移动端隐藏(移动端详情入口仍为双击/长按菜单);
    • 行内编辑态保留占位、隐藏按钮,避免布局跳动。
  • 顺带修复一个真机才暴露的 CSS 特异性缺陷:行悬停规则 .group/task-row:hover .gantt-row-open-btn(0,3,0)压过按钮自身 :hover { opacity: 1 }(0,2,0),按钮自悬停时永远停在 60% 透明度——用 :where() 把行悬停规则降为零特异性。

测试

  • 单元测试:plugin-gantt 303 passed(新增断言:定位按钮不渲染、「→」非 absolute 覆盖层),i18n 176 passed,tsc 干净。
  • 真机验证:console 集成构建部署到消费项目(os-tianshun-ehr)后 Playwright 实测 4/4 通过,含截图——见 issue 上的测试报告:定位图标零残留;按钮 24×24、左缘 ≥ 结束列右缘(零重叠);自悬停 opacity=1 + 底色;点击开抽屉;单击定位/双击详情回归通过。

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
objectui Ignored Ignored Jul 14, 2026 6:15am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

✅ Console Performance Budget

Metric Value Budget
Main entry (gzip) 27.5 KB 350 KB
Entry file index-DWSQ3hht.js
Status PASS

📦 Bundle Size Report

Package Size Gzipped
app-shell (index.js) 7.84KB 2.85KB
app-shell (runtime-config.js) 7.42KB 2.32KB
app-shell (types.js) 0.01KB 0.04KB
app-shell (urlParams.js) 7.57KB 2.97KB
auth (AuthContext.js) 0.31KB 0.24KB
auth (AuthGuard.js) 1.17KB 0.53KB
auth (AuthProvider.js) 21.70KB 4.21KB
auth (AuthShell.js) 3.49KB 1.40KB
auth (ForgotPasswordForm.js) 12.12KB 3.41KB
auth (LoginForm.js) 17.12KB 5.00KB
auth (PreviewBanner.js) 0.90KB 0.50KB
auth (RegisterForm.js) 6.63KB 2.15KB
auth (SocialSignInButtons.js) 9.60KB 3.89KB
auth (UserMenu.js) 3.40KB 1.22KB
auth (auth-gate-events.js) 1.29KB 0.66KB
auth (authStyles.js) 5.04KB 1.72KB
auth (createAuthClient.js) 30.17KB 7.28KB
auth (createAuthenticatedFetch.js) 3.93KB 1.55KB
auth (index.js) 1.83KB 0.79KB
auth (phone-identifier.js) 1.11KB 0.66KB
auth (types.js) 0.59KB 0.35KB
auth (useAuth.js) 4.86KB 0.85KB
auth (useIsWorkspaceAdmin.js) 1.61KB 0.85KB
collaboration (CommentThread.js) 18.38KB 4.49KB
collaboration (LiveCursors.js) 3.17KB 1.27KB
collaboration (PresenceAvatars.js) 3.65KB 1.42KB
collaboration (PresenceProvider.js) 2.42KB 0.96KB
collaboration (index.js) 1.25KB 0.53KB
collaboration (useCommentSearch.js) 1.98KB 0.88KB
collaboration (useConflictResolution.js) 7.75KB 1.86KB
collaboration (useMentionNotifications.js) 1.81KB 0.68KB
collaboration (usePresence.js) 6.33KB 1.84KB
collaboration (useRealtimeSubscription.js) 7.91KB 2.01KB
components (index.js) 441.87KB 95.38KB
core (index.js) 1.65KB 0.59KB
create-plugin (index.js) 9.28KB 2.98KB
data-objectstack (index.js) 113.70KB 28.15KB
fields (index.js) 202.54KB 49.50KB
i18n (LocalizationContext.js) 1.76KB 0.96KB
i18n (currency.js) 1.22KB 0.64KB
i18n (i18n.js) 4.32KB 1.77KB
i18n (index.js) 2.46KB 0.96KB
i18n (pickLocalized.js) 1.31KB 0.67KB
i18n (provider.js) 5.37KB 1.72KB
i18n (useObjectLabel.js) 21.15KB 4.68KB
i18n (useSafeTranslation.js) 2.68KB 0.98KB
layout (index.js) 38.45KB 10.67KB
mobile (MobileProvider.js) 0.92KB 0.49KB
mobile (ResponsiveContainer.js) 0.94KB 0.38KB
mobile (breakpoints.js) 1.51KB 0.70KB
mobile (createOfflineDataSource.js) 5.61KB 1.74KB
mobile (index.js) 1.50KB 0.62KB
mobile (offlineQueue.js) 3.91KB 1.35KB
mobile (pwa.js) 0.97KB 0.49KB
mobile (serviceWorker.js) 1.48KB 0.62KB
mobile (serviceWorkerSource.js) 3.41KB 1.48KB
mobile (useBreakpoint.js) 1.54KB 0.65KB
mobile (useGesture.js) 4.42KB 1.27KB
mobile (useOfflineSync.js) 1.99KB 0.72KB
mobile (usePullToRefresh.js) 2.53KB 0.85KB
mobile (useResponsive.js) 0.71KB 0.42KB
mobile (useResponsiveConfig.js) 1.36KB 0.63KB
mobile (useSpecGesture.js) 1.77KB 0.77KB
mobile (useTouchTarget.js) 1.01KB 0.54KB
permissions (MePermissionsProvider.js) 5.09KB 1.84KB
permissions (PermissionContext.js) 0.31KB 0.25KB
permissions (PermissionGuard.js) 0.89KB 0.45KB
permissions (PermissionProvider.js) 3.46KB 1.03KB
permissions (evaluator.js) 4.00KB 1.23KB
permissions (index.js) 0.91KB 0.41KB
permissions (store.js) 0.91KB 0.42KB
permissions (useFieldPermissions.js) 1.28KB 0.52KB
permissions (usePermissions.js) 1.50KB 0.70KB
plugin-ai (index.js) 15.71KB 3.79KB
plugin-calendar (index.js) 45.12KB 12.34KB
plugin-charts (index.js) 46.31KB 13.06KB
plugin-chatbot (index.js) 176.18KB 42.05KB
plugin-dashboard (index.js) 108.35KB 26.87KB
plugin-designer (index.js) 213.94KB 43.03KB
plugin-detail (index.js) 210.11KB 50.70KB
plugin-editor (index.js) 2.46KB 1.10KB
plugin-form (index.js) 104.26KB 25.24KB
plugin-gantt (index.js) 149.17KB 37.17KB
plugin-grid (index.js) 172.19KB 45.48KB
plugin-kanban (index.js) 48.16KB 12.94KB
plugin-list (index.js) 98.74KB 23.30KB
plugin-map (index.js) 16.80KB 5.24KB
plugin-markdown (index.js) 13.65KB 4.67KB
plugin-report (index.js) 37.82KB 9.96KB
plugin-timeline (index.js) 25.37KB 7.20KB
plugin-tree (index.js) 8.21KB 2.76KB
plugin-view (index.js) 85.51KB 20.81KB
providers (DataSourceProvider.js) 0.75KB 0.39KB
providers (MetadataProvider.js) 1.37KB 0.59KB
providers (ThemeProvider.js) 1.55KB 0.67KB
providers (UploadProvider.js) 11.71KB 3.53KB
providers (index.js) 0.44KB 0.22KB
providers (types.js) 0.01KB 0.04KB
react-runtime (index.js) 3.19KB 1.38KB
react (LazyPluginLoader.js) 3.77KB 1.33KB
react (SchemaRenderer.js) 18.23KB 5.97KB
react (data-invalidation.js) 5.05KB 2.08KB
react (index.js) 0.79KB 0.43KB
sdui-parser (codegen.js) 4.09KB 1.74KB
sdui-parser (index.js) 2.16KB 0.94KB
sdui-parser (parse.js) 10.04KB 2.82KB
sdui-parser (types.js) 0.29KB 0.24KB
sdui-parser (validate.js) 4.69KB 1.48KB
tenant (TenantContext.js) 0.31KB 0.25KB
tenant (TenantGuard.js) 1.04KB 0.43KB
tenant (TenantProvider.js) 2.76KB 0.98KB
tenant (TenantScopedQuery.js) 0.77KB 0.44KB
tenant (index.js) 0.75KB 0.38KB
tenant (resolver.js) 2.64KB 0.76KB
tenant (useTenant.js) 0.50KB 0.32KB
tenant (useTenantBranding.js) 0.62KB 0.39KB
types (ai.js) 0.20KB 0.17KB
types (api-types.js) 0.20KB 0.18KB
types (app.js) 2.87KB 0.99KB
types (base.js) 0.20KB 0.18KB
types (blocks.js) 0.20KB 0.18KB
types (complex.js) 0.20KB 0.18KB
types (crud.js) 0.20KB 0.18KB
types (data-display.js) 0.20KB 0.18KB
types (data-protocol.js) 0.20KB 0.19KB
types (data.js) 0.20KB 0.18KB
types (designer.js) 0.77KB 0.41KB
types (disclosure.js) 0.20KB 0.18KB
types (feedback.js) 0.20KB 0.18KB
types (field-types.js) 0.20KB 0.18KB
types (form.js) 0.20KB 0.18KB
types (index.js) 1.54KB 0.68KB
types (layout.js) 0.20KB 0.18KB
types (mobile.js) 0.20KB 0.18KB
types (navigation.js) 0.20KB 0.18KB
types (objectql.js) 0.20KB 0.18KB
types (overlay.js) 0.20KB 0.18KB
types (permissions.js) 0.20KB 0.18KB
types (plugin-scope.js) 0.20KB 0.18KB
types (record-components.js) 0.20KB 0.19KB
types (registry.js) 0.20KB 0.18KB
types (reports.js) 0.20KB 0.18KB
types (spec-report.js) 5.26KB 1.96KB
types (tenant.js) 0.20KB 0.18KB
types (theme.js) 0.20KB 0.18KB
types (ui-action.js) 0.75KB 0.46KB
types (views.js) 0.20KB 0.18KB
types (widget.js) 0.20KB 0.18KB

Size Limits

  • ✅ Core packages should be < 50KB gzipped
  • ✅ Component packages should be < 100KB gzipped
  • ⚠️ Plugin packages should be < 150KB gzipped

@baozhoutao baozhoutao merged commit 694f67f into main Jul 14, 2026
10 checks passed
@baozhoutao baozhoutao deleted the claude/gantt-row-open-ux-2482 branch July 14, 2026 06:32
os-zhuang added a commit that referenced this pull request Jul 14, 2026
…asts (#2488)

The AI chat switcher/bound-package derivations cast `(app as { _packageId?: string })`
at every access (4 sites). Narrow `useMetadata().apps` once to a local
MetadataAppItem shape at the hook boundary; call sites now read .name / .label /
._packageId without casts. Type-only, no runtime change.

Refs #2487 (A1).

Co-authored-by: Jack Zhuang <277994282+os-zhuang@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@os-zhuang

Copy link
Copy Markdown
Contributor

Progress:

Remaining items (A2–A4, B1–B3, C1–C4) intentionally NOT batch-rammed in one pass: B changes behavior (needs live verification), C is large/cross-repo (needs review + cloud coordination). Recommend proceeding one PR at a time with real CI + live checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

甘特图任务行操作按钮体验:删除冗余「定位」悬停图标,「→」详情按钮改独立操作槽(现被结束列压住、热区过小)

2 participants