Skip to content

fix: domain picker overflow, triage skip-for-today, important/urgent at creation#179

Open
meninoebom wants to merge 1 commit into
mainfrom
fix-domain,-triage,-priority-bugs
Open

fix: domain picker overflow, triage skip-for-today, important/urgent at creation#179
meninoebom wants to merge 1 commit into
mainfrom
fix-domain,-triage,-priority-bugs

Conversation

@meninoebom
Copy link
Copy Markdown
Owner

Summary

Three reported bugs fixed:

  • Domain picker no longer clips Work / None off-screen. The dropdown was centered under the trigger dot with left-1/2 -translate-x-1/2; on narrow viewports the leftmost half overflowed. Reanchored to left-0 with a viewport-width guard. (domain-picker.tsx)
  • Triage reminder banner can now be skipped for the day. "Maybe later" only delayed the banner until the next nav. Added a third "Skip for today" button that calls the existing markTriageDone() (sessionStorage), so checkTriage() short-circuits on subsequent navigation and reload. "Maybe later" still does the old delay-only behavior. (triage-reminder-banner.tsx, (app)/layout.tsx)
  • Important / urgent can be set at task creation. Backend already accepted these on TaskCreate; frontend just didn't surface them. Added two pill-style toggle buttons in TaskInput matching the styling of the in-list priority pills, plus the optional fields on the createTask body type. State resets after each create; existing keyboard flow (Enter → chips → Enter) preserved. (task-input.tsx, lib/api.ts)

No backend changes. Five files touched. Lint + build clean.

Test plan

  • Narrow viewport (≤ 400px / iPhone SE): open the domain dot picker on a task — all six options ("None", Work, Personal, Health, Creative, Admin) visible
  • With un-triaged tasks: banner appears on /today; click "Skip for today"; navigate to /bucket/later then back to /today — banner does NOT return; reload page same day — still no banner
  • "Maybe later" still has old behavior (banner returns on next nav)
  • Type a new task; toggle ! and ⚡; submit — created task shows both pills filled and gets the Q1 red left-border in the list
  • After creation, toggles in TaskInput reset to off
  • Existing keyboard flow Enter → chips → Enter still works

🤖 Generated with Claude Code

…at creation

- Domain picker: anchor dropdown to left-0 instead of centered translate
- Triage banner: add "Skip for today" button using existing markTriageDone()
- TaskInput: add !/⚡ priority toggle pills; pass important/urgent on create

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@railway-app
Copy link
Copy Markdown

railway-app Bot commented May 2, 2026

🚅 Deployed to the tend-pr-179 environment in Tend

Service Status Web Updated (UTC)
Tend Frontend ✅ Success (View Logs) Web May 2, 2026 at 8:25 am
Tend Backend ✅ Success (View Logs) May 2, 2026 at 8:25 am

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.

1 participant