Skip to content

Conversation

@zunaid-hassan
Copy link
Contributor

Clarified instructions for building a dynamic shopping list by removing redundant phrasing about item deletion.

Description

Removes a duplicated requirement (item 2 and 3 were identical) to prevent confusion in the exercise.

Changes:

  • The item should appear in the list.
  • Each item should be given a button that can be pressed to delete that item off the list.
  • Each item should have a button next to it that removes the item from the list when clicked.
  • The input fields should be cleared and focused, ready for the next item entry.

To:

  • The item should appear in the list.
  • Each item should have a button next to it that removes the item from the list when clicked.
  • The input fields should be cleared and focused, ready for the next item entry.

Motivation

Removes unnecessary repetition and makes it easier for learners to progress.

Additional details

Article: https://developer.mozilla.org/en-US/docs/Learn_web_development/Core/Scripting/DOM_scripting#creating_a_dynamic_shopping_list

Related issues and pull requests

N/A

Clarified instructions for building a dynamic shopping list by removing redundant phrasing about item deletion.
@zunaid-hassan zunaid-hassan requested a review from a team as a code owner November 1, 2025 09:13
@zunaid-hassan zunaid-hassan requested review from estelle and removed request for a team November 1, 2025 09:13
@github-actions github-actions bot added Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed labels Nov 1, 2025
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks

@Josh-Cena Josh-Cena merged commit 273e96b into mdn:main Nov 1, 2025
7 checks passed
@zunaid-hassan zunaid-hassan deleted the patch-1 branch November 2, 2025 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants