-
Notifications
You must be signed in to change notification settings - Fork 41.4k
Open
Labels
Status: Needs ReviewThis issue/PR needs an initial or additional reviewThis issue/PR needs an initial or additional review
Description
Prerequisites
- I have thoroughly read and understand The Odin Project Contributing GuideI have searched the existing issues to ensure this improvement hasn't been suggested before (please have a look through our open issues list to make sure)The title of this issue follows the
<Location of the improvement>: <Brief description of the improvement>
format, e.g.Exercises: Add exercise on XYZ
(Optional) I am interested in working on this issue and would like to be assigned to itTo pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
Improvement Description
As a TOP student I successfully did this exercise by using two for loops – though it was not pretty.
After checking the solution, I see that both array.forEach
and array.Includes()
would make this a whole lot easier, and I'm thankful I learned that.
But I can't help but think that it would be a better learning experience if I would get a hint about those methods beforehand in the hints section.
I'm pretty sure I wasn't exposed to the either forEach
or Includes
earlier in the course, but I might be wrong.
Acceptance Criteria
- Include a subtle hint about the
forEach
andIncludes
methods in the hints section.To pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
(Optional) Additional Comments
No response
Metadata
Metadata
Assignees
Labels
Status: Needs ReviewThis issue/PR needs an initial or additional reviewThis issue/PR needs an initial or additional review
Activity
eyad-alkhalidy commentedon May 31, 2025
We've actually been exposed to these Array methods earlier, in the javascript.info guide linked here
It's still helpful to encourage learners to use array methods here, but without specifying which ones.