Skip to content

Edit a clock entry's description from its popup#240

Merged
ramonski merged 1 commit into
masterfrom
fix/editable-description-popup
Jul 8, 2026
Merged

Edit a clock entry's description from its popup#240
ramonski merged 1 commit into
masterfrom
fix/editable-description-popup

Conversation

@ramonski

@ramonski ramonski commented Jul 8, 2026

Copy link
Copy Markdown
Member

Context

In the Clocks table, a long clock-entry description opened a read-only expand viewer (ContentPopup, the diagonal-arrows icon), while the notes in the same row opened the editable NotesBubble. Two indicators side by side that behaved differently.

Fix

Point the long-description indicator at the same editable NotesBubble (text-icon variant). Clicking it now opens the Preview/Write dialog and saves the description via updateClockEntry, matching the notes bubble next to it. The icon shows only when the description is long, as before.

Acceptance

  • A long description opens an editable Preview/Write popup and saves.
  • Description (text icon) and notes (speech bubble) in a row are visually distinct but behave identically.
  • tsc + pnpm build clean.

In the Clocks table a long description opened a read-only expand
viewer (ContentPopup), while notes opened the editable NotesBubble --
so the two indicators in the same row behaved differently. Point the
description at the same editable NotesBubble (text-icon variant) so it
opens the Preview/Write dialog and saves the description, matching the
notes bubble next to it.
@ramonski ramonski added the Bug 🐞 Something is broken label Jul 8, 2026
@ramonski ramonski self-assigned this Jul 8, 2026
@ramonski ramonski merged commit 2e18829 into master Jul 8, 2026
2 checks passed
@ramonski ramonski deleted the fix/editable-description-popup branch July 8, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug 🐞 Something is broken

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant