Andras/BoundlessFlowCampus2K
3
6
Fork 0
Code Issues 140 Pull requests 16 Projects Releases Packages Wiki Activity Actions 2

#937-Personal-to-do-list #995

Merged
a22erigr merged 10 commits from #937-Personal-to-do-list into team_2_week_6 2026-05-20 07:12:37 +00:00
Conversation 4 Commits 10 Files changed 4 +260
a24danwe commented 2026-05-18 11:58:45 +00:00
Collaborator
Copy link

Created a ToDoView. It makes a checklist of your created tasks which a progress bar that gives you a visual indication of how much you have completed.

Created a ToDoView. It makes a checklist of your created tasks which a progress bar that gives you a visual indication of how much you have completed.
This is a new view which is intended to be used as a checklist. You create a task and then you can mark it as complete when you are done. When you complete a task the progress bar will go up.
This adds a description to the card so you can read what it does before adding it.
This makes it so the view is visible.
Creates styling for the ToDoView, including the progress bar and task layout.
a24danwe commented 2026-05-18 12:01:28 +00:00
Author
Collaborator
Copy link

Issue #937

Issue #937
a22erigr 2026-05-19 06:56:07 +00:00
a24nedka commented 2026-05-19 08:51:26 +00:00
Collaborator
Copy link

Review on #995

Whats done

  • Reviewed implementation of the new personal to-do list view and progress tracking functionality

Tested on Windows using Firefox

  • App starts
  • No errors detected
  • Verified the new ToDoView renders correctly
  • Tested adding new tasks.
  • Tested removing tasks.
  • Tested checking/unchecking tasks.
  • Verified progress bar updates correctly depending on completed tasks.
  • No UI braking issues found.

Inspected following files : ToDoView.js, ToDoView.module.css, Dashboard.js, tooltipConfig.js, AddCardModal.js

  • All inspected files generally follow the project JavaScript/React and CSS coding conventions.
  • Naming conventions is correctly used.
  • Formatting, spacing, indentation, and comments are overall consistent with project standards.
  • The new ToDoView integrates cleanly into the existing dashboard structure and follows existing architectural patterns.
  • CSS modules are structured clearly with descriptive comments and reusable layouts.
  • Small issues found see down below what needs to be fixed.

What needs to be done

  • Need to fix the button add so it can be seen in darkmode. See screenshot down below.
  • Issue abow can be fixed with changing hardcoded color to follow theme.

Conclusion

  • Not ready for merge before the issue is fixed.

imageimage

# Review on #995 ## Whats done + Reviewed implementation of the new personal to-do list view and progress tracking functionality ### Tested on Windows using Firefox + App starts + No errors detected + Verified the new ToDoView renders correctly + Tested adding new tasks. + Tested removing tasks. + Tested checking/unchecking tasks. + Verified progress bar updates correctly depending on completed tasks. + No UI braking issues found. ### Inspected following files : ToDoView.js, ToDoView.module.css, Dashboard.js, tooltipConfig.js, AddCardModal.js + All inspected files generally follow the project JavaScript/React and CSS coding conventions. + Naming conventions is correctly used. + Formatting, spacing, indentation, and comments are overall consistent with project standards. + The new ToDoView integrates cleanly into the existing dashboard structure and follows existing architectural patterns. + CSS modules are structured clearly with descriptive comments and reusable layouts. + Small issues found see down below what needs to be fixed. ## What needs to be done + Need to fix the button add so it can be seen in darkmode. See screenshot down below. + Issue abow can be fixed with changing hardcoded color to follow theme. ## Conclusion + Not ready for merge before the issue is fixed. ![image](/attachments/691f7c14-28c2-4844-93ab-11cdd7b229e5)![image](/attachments/8a6a516a-ec39-45c6-a657-30cdea5c7c86)
image.png
24 KiB
image.png
23 KiB
a24danwe commented 2026-05-19 17:11:07 +00:00
Author
Collaborator
Copy link

Thank you for the review!

I have improved the hardcoded colors and replaced it with appropriate colors from theme.css in frontend/src/app/theme.css

I think this is better even though for example the progress bar is more of a dark purple than a green color.

Thank you for the review! I have improved the hardcoded colors and replaced it with appropriate colors from theme.css in frontend/src/app/theme.css I think this is better even though for example the progress bar is more of a dark purple than a green color.
a22erigr commented 2026-05-20 07:12:04 +00:00
Collaborator
Copy link

bild

  • The card now follows the color scheme
  • The buttons are not visible. This will probably be fixed after our new card design. There by the issue is seemed to be fit to merge.
![bild](/attachments/09140c5f-6c26-445b-a19b-f481008c7be5) + The card now follows the color scheme + The buttons are not visible. This will probably be fixed after our new card design. There by the issue is seemed to be fit to merge.
bild.png
14 KiB
a22erigr merged commit ac5a2e41e1 into team_2_week_6 2026-05-20 07:12:37 +00:00
a22erigr deleted branch #937-Personal-to-do-list 2026-05-21 12:40:01 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
a24nedka
Labels
Clear labels   
_CRITICAL_

Only use this label for critical tasks that are project-scope,and must be addressed ASAP.

  
API

   
app

   
backEnd

   
Blocked-waiting-for-further-changes

   
bug

Something is not working

  
bug-only-on-server-for-mobile-not-webpage

   
Bug-Report-After-Merge

   
cleanup

Issues to cleanup the codebase (restructure folders, remove deprecated or smelly code)

  
close

Issue to close

  
design

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
feature request

   
frontEnd

   
further-changes-needed

   
future-problem-not-fixint-this-period

   
help wanted

Need some help

  
invalid

Something is wrong

  
last-week-issue-to-fix

   
library

   
low-priority

   
needs input

Issue needs input from product owner or customer

  
needs review

issue is in need of review

  
not-implemented.

   
project documentation

   
question

More information is needed

  
research

   
reviewed

This Issue has been reviewed

  
Script

   
security

use this for security-related issues

  
SQL

  
style

   
Team 1

Team 1 is responsible for this issue

  
Team 2

Team 2 is responsible for this issue

  
team leaders

   
test-creation

   
testing

   
topLevel

Reserved for top-level, project-wide issues.

  
unassigned

   
Under-review

   
wontfix

This won't be fixed

No labels
_CRITICAL_
API
app
backEnd
Blocked-waiting-for-further-changes
bug
bug-only-on-server-for-mobile-not-webpage
Bug-Report-After-Merge
cleanup
close
design
duplicate
enhancement
feature request
frontEnd
further-changes-needed
future-problem-not-fixint-this-period
help wanted
invalid
last-week-issue-to-fix
library
low-priority
needs input
needs review
not-implemented.
project documentation
question
research
reviewed
Script
security
SQL
style
Team 1
Team 2
team leaders
test-creation
testing
topLevel
unassigned
Under-review
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
a24danwe
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!995
No description provided.