Skip to content

Add hover remove control to message tags - #264

Draft
OJFord with Copilot wants to merge 2 commits into
masterfrom
copilot/show-x-to-remove-tag-on-hover
Draft

Add hover remove control to message tags#264
OJFord with Copilot wants to merge 2 commits into
masterfrom
copilot/show-x-to-remove-tag-on-hover

Conversation

Copilot AI commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

This updates tag badges in the list and reader views so individual tags can be removed in place via a hover affordance, instead of requiring bulk tag actions. The change keeps tag editing local to the tag being acted on and refreshes surrounding state after removal.

  • UI behavior

    • Adds a hover-only × control to removable tag badges.
    • Keeps the control attached to the badge itself so the interaction is visible only when relevant.
  • Tag removal wiring

    • Passes the message-scoped query (id:<message-id>) into tag badge rendering.
    • Calls the existing rmTag API for the selected tag and emits tagsUpdated on success.
  • State propagation

    • Updates both the reader pane and message list item to remove the tag locally after success.
    • Bubbles tagsUpdated back up so query/tag lists refresh consistently.
  • Interaction safety

    • Prevents the remove control from triggering parent click handlers in the message list.
<TagBadges
  tags={emlMeta.tags}
  query={`id:${emlMeta.id}`}
  on:tagsUpdated={(event) => {
    emlMeta.tags = emlMeta.tags.filter((tag) => tag != event.detail.tag)
    dispatch("tagsUpdated", event.detail)
  }}
/>

Copilot AI linked an issue Jun 7, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add 'x' to remove tag on hover Add hover remove control to message tags Jun 7, 2026
Copilot finished work on behalf of OJFord June 7, 2026 20:38
Copilot AI requested a review from OJFord June 7, 2026 20:38
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.

Show 'x' to remove tag on hover

2 participants