feat: Redesign Tasks page with simplified categories and header controls #3948
Conversation
- Simplified task categories to: Today, No Deadline, Later - Added drag-and-drop between categories (updates task deadline) - Added swipe left/right to indent tasks (cascade like Apple Notes) - Moved export and completed toggle buttons to header bar - Removed banner and tabs UI for cleaner single-page layout - Added 7-day filter to hide old uncompleted tasks - Synced changes to desktop version 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a significant and well-executed redesign of the Tasks page for both mobile and desktop platforms. The new design simplifies task categories, introduces intuitive drag-and-drop and swipe-to-indent gestures, and provides a cleaner user interface by moving controls to the header. The code is well-structured, and the changes are consistently applied across both platforms.
My main feedback is regarding a repeated violation of a repository rule about using a deprecated method for color opacity. I've pointed out specific instances and recommended updating them to maintain code consistency and adhere to the established best practices.
| end: Alignment.bottomCenter, | ||
| colors: [ | ||
| ResponsiveHelper.backgroundPrimary, | ||
| ResponsiveHelper.backgroundSecondary.withOpacity(0.8), |
There was a problem hiding this comment.
According to the repository's general rules, the withOpacity() method is deprecated and withValues({double? alpha}) should be used instead to modify a color's opacity. This ensures consistency with the project's coding standards. I've noticed several other instances of withOpacity() in this file (e.g., lines 264, 405, 527, 588, 723, and 766) that should also be updated.
| ResponsiveHelper.backgroundSecondary.withOpacity(0.8), | |
| ResponsiveHelper.backgroundSecondary.withValues(alpha: 0.8), |
References
- Use the custom
withValues({double? alpha})extension method instead of the deprecatedwithOpacity()to modify a color's opacity.
| decoration: BoxDecoration( | ||
| color: item.completed ? Colors.orange : Colors.green, | ||
| borderRadius: BorderRadius.circular(16), | ||
| color: Colors.deepPurpleAccent.withOpacity(0.1), |
There was a problem hiding this comment.
The repository's general rules specify that withValues({double? alpha}) should be used instead of the deprecated withOpacity() method. Please update this to adhere to the project's coding standards. This also applies to other occurrences in this file, such as on lines 241 and 386.
| color: Colors.deepPurpleAccent.withOpacity(0.1), | |
| color: Colors.deepPurpleAccent.withValues(alpha: 0.1), |
References
- Use the custom
withValues({double? alpha})extension method instead of the deprecatedwithOpacity()to modify a color's opacity.
Summary
Test plan
🤖 Generated with Claude Code