Skip to content

Improve edit UI and priority colors - #1177

Merged
dongdongbh merged 7 commits into
dongdongbh:mainfrom
Kombustor:feature/improve-edit-ui-and-priority-colors
Sep 6, 2026
Merged

Improve edit UI and priority colors#1177
dongdongbh merged 7 commits into
dongdongbh:mainfrom
Kombustor:feature/improve-edit-ui-and-priority-colors

Conversation

@Kombustor

Copy link
Copy Markdown
Contributor

Summary

Improves task-editor clarity with consistent vector icons, colored priority flags, labelled icon-only controls, and cleaner energy labels across desktop and mobile. Adds a mobile close control wired through the existing unsaved-changes flow while preserving Save behavior and accessibility labels.

Related issue

Closes #1172

Testing

Full mobile & desktop test suite.

Screenshots / recordings

Bildschirmfoto 2026-09-06 um 21 07 18 Bildschirmfoto 2026-09-06 um 21 09 34 Bildschirmfoto 2026-09-06 um 21 09 12 Bildschirmfoto 2026-09-06 um 21 08 59 Screenshot_1788722213 Screenshot_1788722218 Screenshot_1788722225 Screenshot_1788722227

Checklist

  • I have signed the Contributor License Agreement (CLA)
  • I have tested this change locally
  • I linked the relevant issue (or explained why there isn't one)
  • I added or updated tests/docs if needed

…top and mobile and add a feature-gated Priority… action to the task quick menu, centralizing the fixed ramp in core so row strips, editor pills, slash commands, filters, inbox processing and quick capture share one affordance
…turn Preview/Edit, change-time, clear and attachment row actions into labelled icon buttons, and align attachment and checklist button styling
…urn change-time, clear, preview, edit and attachment actions into labelled icon buttons, make None controls icon-only, and align attachment and Add Item button styling
…None and swap the change-time chip for a labelled clock icon button
@Kombustor Kombustor changed the title Feature/improve edit UI and priority colors Improve edit UI and priority colors Sep 6, 2026
@Kombustor Kombustor mentioned this pull request Sep 6, 2026
14 tasks
Kombustor and others added 2 commits September 6, 2026 21:29
…ity colours from core everywhere, swap the last text glyphs for lucide icons and stub every lucide icon in tests (dongdongbh#1172)

@dongdongbh dongdongbh left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready to merge

Good work, this lands the whole of #1172 and the Priority quick action fits the other quick actions well, so it stays.

I pushed one commit on top of your branch (95fe85c) rather than asking for another round:

  • The mobile editor header shows the word Save again. The X on the left is fine, but a first-time user needs to see which control commits the draft, so Save keeps its text. The header test now asserts that.
  • The mobile widget now reads TASK_PRIORITY_COLORS from core and apps/mobile/lib/priority-colors.ts is gone, so there is one priority ramp for the app and the widget. The unused import in SwipeableTaskItemContent.tsx went with it.
  • The lucide mock in apps/mobile/vitest.setup.ts now stubs any icon name instead of an allowlist. Navigation and about fifteen older imports were missing from the list, and the first test to render one of those fields would have failed far from the change.
  • The last text glyphs on the lines you touched are lucide icons now: the sparkle on the desktop "Create project" option, the plus on the mobile Add item and picker create rows, and the check marks in the quick capture pickers.

@dongdongbh
dongdongbh merged commit e52e6d8 into dongdongbh:main Sep 6, 2026
16 checks passed
@dongdongbh

Copy link
Copy Markdown
Owner

Ready to merge

Good work, this lands the whole of #1172 and the Priority quick action fits the other quick actions well, so it stays.

I pushed one commit on top of your branch (95fe85c) rather than asking for another round:

  • The mobile editor header shows the word Save again. The X on the left is fine, but a first-time user needs to see which control commits the draft, so Save keeps its text. The header test now asserts that.
  • The mobile widget now reads TASK_PRIORITY_COLORS from core and apps/mobile/lib/priority-colors.ts is gone, so there is one priority ramp for the app and the widget. The unused import in SwipeableTaskItemContent.tsx went with it.
  • The lucide mock in apps/mobile/vitest.setup.ts now stubs any icon name instead of an allowlist. Navigation and about fifteen older imports were missing from the list, and the first test to render one of those fields would have failed far from the change.
  • The last text glyphs on the lines you touched are lucide icons now: the sparkle on the desktop "Create project" option, the plus on the mobile Add item and picker create rows, and the check marks in the quick capture pickers.

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.

[FEAT] Improve task-editing UI

2 participants