Skip to content

Move copy task ID functionality to context menu#1558

Merged
adboio merged 1 commit intomainfrom
posthog-code/remove-task-id-display-from-taskdetail-component-2
Apr 8, 2026
Merged

Move copy task ID functionality to context menu#1558
adboio merged 1 commit intomainfrom
posthog-code/remove-task-id-display-from-taskdetail-component-2

Conversation

@adboio
Copy link
Copy Markdown
Contributor

@adboio adboio commented Apr 8, 2026

TL;DR

Moved the "Copy Task ID" feature from the TaskDetail header to the task context menu, removing the task ID display from the component header and centralizing this action in the context menu.

Problem

The task ID was displayed as a clickable button in the TaskDetail header, taking up valuable space. This functionality is better served as a context menu action that's available on-demand.

Changes

  • Added "Copy Task ID" context menu action to task actions in context-menu/schemas.ts and context-menu/service.ts
  • Removed task ID display from TaskDetail component header - no longer shows the task ID button in the header UI
  • Moved copy functionality from TaskDetail component to useTaskContextMenu hook where it handles the clipboard write and toast notification
  • Cleaned up TaskDetail imports - removed unused Tooltip from radix-ui, useCallback from React, and toast from sonner (now only imported in the hook)
  • Simplified header content memoization - removed task ID and copyTaskId from dependency array

How did you test this?

  • The changes are based on moving existing functionality to a new location rather than adding new behavior
  • The copy functionality logic remains the same (clipboard write + success toast)
  • The context menu integration follows the existing pattern for task actions

Created with PostHog Code

Generated-By: PostHog Code
Task-Id: f9d7c635-6d95-4255-8a4d-d2281b1e0389
@adboio adboio force-pushed the posthog-code/remove-task-id-display-from-taskdetail-component-2 branch from 65c446e to 06b6d63 Compare April 8, 2026 18:47
Copy link
Copy Markdown
Contributor Author

adboio commented Apr 8, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@adboio adboio merged commit d113d62 into main Apr 8, 2026
16 checks passed
Copy link
Copy Markdown
Contributor Author

adboio commented Apr 8, 2026

Merge activity

@adboio adboio deleted the posthog-code/remove-task-id-display-from-taskdetail-component-2 branch April 8, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants