-
Notifications
You must be signed in to change notification settings - Fork 75
Refine Add Task dialog layout and advanced fields #444
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Refine Add Task dialog layout and advanced fields #444
Conversation
|
Thank you for opening this PR! Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools. Please take a moment to:
More information on how to conduct a self review: This helps make the review process smoother and gives us a clearer understanding of your thought process. Once you've added your self-review, we'll continue from our side. Thank you! |
Rustix69
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Redesigned the AddTaskDialog into a wider, scrollable layout that groups fields into clear sections: Essential Fields, Dates, and Advanced Fields, while keeping all existing behavior intact. The Due Date now clearly supports time, and advanced fields (tags, annotations, dependencies, recurrence) have more structured, visually distinct UIs with icons, counts, and better spacing. Tests were updated to match the new DOM structure so they still validate the same behaviors against the refreshed UI.
|
@its-me-abhishek Hey I had added the self review and all comments. |
its-me-abhishek
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rustix69 this looks great! The value cards for Annotations looks great, could you please do the same for tags? Also, I think we should stick to the tags-like view to show cards, instead of new headings like "Selected Annotations", "Selected Headings", etc. so as to save space.
|
@its-me-abhishek Now it looks more sleek thnks for the suggestion.
|
its-me-abhishek
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix these, moreover do add the latest desktop/mobile view screenshots for both Dark/Light Mode to the PR description!
|
@its-me-abhishek I had removed the comments also added the Dark/Light View in the Description of the PR. |

Description
Improve the Add Task dialog UX in the Tasks view by widening the dialog, grouping related fields into clearly labeled sections, and refreshing the styling of advanced fields. The new layout keeps all fields visible in a single scrollable panel (no accordion), uses Cards for “Essential Fields”, “Dates”, and “Advanced Fields”, and enhances tags/annotations/dependencies/recurrence with clearer visuals and counts.
Checklist
npx prettier --write .(for formatting)gofmt -w .(for Go backend)npm test(for JS/TS testing)Screenshot
Light Mode
Dark Mode
Mobile View
Additional Notes