Feat: Add celebration animation on completing all tasks in Daily Planner (#341) #350
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Implement task editing functionality with save/cancel options - Replace index-based task management with UUID keys - Add session state variables for editing mode - Improve task deletion to avoid re-indexing issues
Show balloons and success message when all tasks are marked as completed to provide positive reinforcement
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces a celebration feature in the Daily Planner section of the Wellness Resource Hub, addressing issue #341.
Changes Introduced
st.balloons()) when all tasks are completed.st.success()) to acknowledge daily achievement.Technical Details
Celebration is triggered when:
completed_count == total_counttotal_count > 0(ensures no celebration for empty lists).Logic is integrated into the existing task display block using already computed task counts.
Screenshot
Linked Issue
Closes #341