DOM Manipulation and Events: More clearly describe the relationship between nodes and elements. about curriculum HOT 4 OPEN

@TheOdinProject/javascript Can someone please review.

from curriculum.

@Hachibroku do you feel even the assigned resource - MDN's DOM Manipulation - doesn't exactly satisfy going over the difference between the two?

If not, then adding a short blurb may not be a bad idea. We should try to keep it succinct, though, just due to how much content the lesson is already covering. Depending on your thoughts to the above question, we can move forward from there.

from curriculum.

@thatblindgeye After re-reading the materials and improving my own understanding of the content, I think that one should be able to understand how 'element' and 'node' relate to each other, but it is somewhat obscured by the lesson. Of the resources linked in the lesson, Digital Ocean - Understanding the DOM Tree and Nodes contained the explanation I was looking for.

I think directly addressing node types and how we're using them would improve the clarity of the lesson overall. For example, an appendage to the paragraph under the Document Object Model heading so it now reads;

The DOM (or Document Object Model) is a tree-like representation of the contents of a webpage - a tree of “nodes” with different relationships depending on how they’re arranged in the HTML document. There are many types of node, most of which are not commonly used. In this lesson we will be focusing on "element" nodes which are primarily used for manipulating the DOM.

or the same information in a separate, clarifying blurb may work better than trying to integrate into the existing content.

I think it may also help to change the final sentence in the paragraph under the DOM Methods heading to say

those that help you target elements.

instead of

those that help you target nodes.

since query selectors specifically work with elements, and it reinforces the node type.

from curriculum.

I think your tweak to the first paragraph sounds good. I'd also be fine with your other suggestion, though we could also just remove that sentence entirely since it doesn't really feel all that necessary to say "First we'll cover [thing]. (heading for [thing] on the very next line)".

Assigned you to the issue, though!

from curriculum.