Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with "Getting started with Svelte": … #1501

Closed
davidecr2010 opened this issue Jan 19, 2021 · 3 comments
Closed

Issue with "Getting started with Svelte": … #1501

davidecr2010 opened this issue Jan 19, 2021 · 3 comments
Labels
Content:Learn Learning area docs good first issue A good issue for newcomers to get started with. help wanted If you know something about this topic, we would love your help! p1 Major issue with medium priority, should be addressed soon.

Comments

@davidecr2010
Copy link

MDN URL: https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Svelte_getting_started

What information was incorrect, unhelpful, or incomplete?

The tutorial suggest making change in the section: https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Svelte_getting_started#making_a_couple_of_changes.
And specifies that changes should automatically be available in the running app ("Just save your changes and the app running at localhost:5000 will be automatically updated.")

Specific section or headline?

https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Svelte_getting_started#making_a_couple_of_changes

What did you expect to see?

The change should be reflected automatically as suggested (Hot reloading?) but even after reloading the page in the browser does the content get updated.

Did you test this? If so, how?

Make the changes on the App.svelte file as suggested in the tutorial. After that I checked in the official documentation of Svelte and it seems that this is not still available as per version 3 of Svelte. Hot reloading needs some sort of extra plugin (sveltejs/svelte#2377)

MDN Content page report details
@chrisdavidmills chrisdavidmills added Content:JS JavaScript docs Content:Learn Learning area docs labels Jan 19, 2021
@chrisdavidmills chrisdavidmills added 30 minute task good first issue A good issue for newcomers to get started with. help wanted If you know something about this topic, we would love your help! p1 Major issue with medium priority, should be addressed soon. and removed Content:JS JavaScript docs labels Mar 18, 2021
@chrisdavidmills
Copy link
Contributor

This si quite an important detail to get right. Whoever fixes this would need to get a svelte app running, check the nature of hot reloading by default, and change the article text accordingly if this is not the case.

@himanshugarg
Copy link
Contributor

himanshugarg commented May 19, 2021

Cannot reproduce on Ubuntu 16, Node v14.16.0 with setup as described. Changing the text and saving the file in editor updates the text in the browser window automatically. @chrisdavidmills let me know if you need further details to be able to close this.

@sideshowbarker
Copy link
Collaborator

Closing as not reproducible per #1501 (comment) (thanks @himanshugarg).

@davidecr2010 If you can still reproduce the problem, this issue can be revisited.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Content:Learn Learning area docs good first issue A good issue for newcomers to get started with. help wanted If you know something about this topic, we would love your help! p1 Major issue with medium priority, should be addressed soon.
Projects
None yet
Development

No branches or pull requests

4 participants