-
Notifications
You must be signed in to change notification settings - Fork 68
feat: Add enhanced pin/unpin task feature with improved UX #384
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
feat: Add enhanced pin/unpin task feature with improved UX #384
Conversation
- Add clickable pin icon directly in task row for one-click pin/unpin - Pinned tasks displayed with amber-colored filled pin icon - Unpinned tasks show subtle pin icon on hover - Add Pin/Unpin toggle button in task dialog footer as secondary option - Implement sorting to show pinned tasks at top of list (above overdue tasks) - Pinned tasks work within all filtered views (projects, tags, status, search) - Store pinned task UUIDs in localStorage per user (using hashed keys) - Add responsive mobile layout with 2-row design: * Row 1: ID, Description, Project * Row 2: Tags, Status, Pin button - Increase task row height on mobile for better touch interaction - No backend changes required (frontend-only feature) Addresses PR feedback: - Quick pin/unpin access without opening dialog - Mobile-optimized layout prevents horizontal scrolling - Pin icon clickable with stopPropagation to avoid opening dialog
|
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! |
- One-click pin/unpin toggle in task rows - Pinned tasks stay at top of list across filters - Comprehensive test coverage (39 new tests) - Privacy-preserving storage with hashed keys
|
@its-me-abhishek I had added the comprehensive TCs for the task pinning feature |
frontend/src/components/HomeComponents/Tasks/__tests__/TaskDialog.test.tsx
Show resolved
Hide resolved
|
@its-me-abhishek I had raised 2-3 Prs adn all have teh tests in the TaskDialog.test.tsx so may be any one of the PRs can get merge conflict so let merge teh PR one by one so that i can review the Merge conflicts |
Description
Implements a pin/unpin task feature that allows users to keep important tasks at the top of their list. Pinned tasks remain at the top within all filtered views.
Changes Made
Frontend
tasks-utils.tsfor localStorage managementEditTaskDialogPropsinterface with pin-related propsTasks.tsxFeatures
Testing Video Snapshot
Screencast.from.2026-01-09.04-08-00.webm
Type of Change
Checklist