TheOdinProject / curriculum

The open curriculum for learning web development
https://www.theodinproject.com/
Other
10.12k stars 13.56k forks source link

JavaScript: Classes 2nd Assignment should tell us to skip the "Accessor descriptors" section #29061

Open hbar1st opened 2 weeks ago

hbar1st commented 2 weeks ago

Checks

Describe your suggestion

In the Classes page of the JS curriculum here , the Assignment section, item #2 starts out by saying:

JavaScript.info’s article on Getters and Setters should get you up to speed on “Getters and Setters”,

When we read it though, there is a section titled "Accessor descriptors" which introduces some new concepts that are confusing and without explanation. These are: descriptor, data property vs accessor property, property flags (writable etc). Direct link to this section: https://javascript.info/property-accessors#accessor-descriptors

Perhaps it would be better to suggest to readers to skip this section of the linked article since it delves into things that haven't been covered by TOP yet? (I assume that these are discussed in a future topic that is. If not, then maybe they ought to be?)

Path

Node / JS

Lesson Url

https://www.theodinproject.com/lessons/node-path-javascript-classes

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

MaoShizhong commented 2 weeks ago

Thanks for opening this issue. The "data/accessor property" bit is fine since they're new terms the article is introducing and explaining at the start. The descriptors/flags bit is written as a continuation of previous teaching on it though, so I agree with that bit.

Personally, I don't think it's necessary to dig into property descriptors at this point. They are unlikely to be vital in TOP projects here, so a note can be added to skip that section since it's out of scope. I like the rest of the article though.

I'll add this change to #28947 which will close this issue as well.