Skip to content

London | ITP-May-2025 | Hibo Sharif | Module-Data-Group | Sprint3 -Todo_list #702

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hibosharif202504
Copy link

@hibosharif202504 hibosharif202504 commented Jul 25, 2025

Learners, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

  • I have implemented the populateTodoList() function to loop through the given list of todos. For each todo, I created a list element and added the appropriate text and action icons. The function now dynamically adds each todo item to the page, matching the structure in index.html.
  • I added a button that allows users to delete all completed ToDos with a single click. When clicked, the button triggers a function that loops through the list of ToDos and removes any that are marked as completed. The undo button makes it easy for users to clean up their list and keep it organised.
  • I also implemented an Undo button that restores the most recently deleted completed ToDos. When a user clicks "Delete All Completed," the removed items are stored temporarily. Clicking "Undo" then brings them back to the list. This feature adds flexibility and prevents the accidental loss of essential Todo items.

Questions

Ask any questions you have for your reviewer.

@hibosharif202504 hibosharif202504 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 25, 2025
@MattGoodwin0 MattGoodwin0 added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Aug 5, 2025
@MattGoodwin0 MattGoodwin0 self-requested a review August 6, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants