Netflix / x-element

A dead simple starting point for custom elements.
Apache License 2.0
28 stars 12 forks source link

Documentation: Clarify that computed properties are lazy. #182

Closed Kelketek closed 2 months ago

Kelketek commented 2 months ago

Small documentation change-- clarifies that computed properties are lazily evaluated. Resolves #178.

klebba commented 2 months ago

+1 — thanks @Kelketek — out of curiosity could you share how you came across this project/what you are using it for? Would love to hear more — thanks!

Kelketek commented 2 months ago

@klebba I just started browsing Netflix's frontend repositories, curious what you all were working on. I found this interesting since previous to this point I've been using tools like React and Vue, and I hadn't seen much in terms of serious use of Web Components.

Diving into this project was a good way to learn about Web Components-- I'd not used them before. I'm hoping to have a PR for you soon to resolve #179 as well, or at least greatly mitigate it between a code tweak and additional documentation. I should hopefully have that for you by tomorrow.

klebba commented 2 months ago

Thanks for the insight @Kelketek and it's nice to hear that this project was helpful for you!