Skip to content
This repository has been archived by the owner on Feb 15, 2023. It is now read-only.

Default clearScreen to false #40

Closed
anonimusprogramus opened this issue Apr 22, 2019 · 2 comments
Closed

Default clearScreen to false #40

anonimusprogramus opened this issue Apr 22, 2019 · 2 comments

Comments

@anonimusprogramus
Copy link

Expected Behavior:

Running npm run dev should make both Sirv and Rollup output some infos to screen.

Actual Behavior:

As soon as Sirv output infos, Rollup's infos covering it.

Proposal:

Please add watch: { clearScreen: false } as default in rollup.config.js, so new user can see what port Svelte's running.

I also have submitted this issue in Rollup repo.

@ArtemKolichenkov
Copy link

+1
I can provide some justification:
Quite often when I have an error and want to scroll up to see what's going on I discover the terminal is not responding to scrolling, instead it just outputs bunch of commands I entered previously or key/scroll input codes (as in this gif below, might add the one with commands print when I encounter it again).

terminalbug

This is known issue for rollup and people indeed suggest adding watch: { clearScreen: false }. Don't know if that has any downsides though (never used rollup before Svelte)

This kind of annoying little things are quite important for developer experience. It took me some time to google around and find what is wrong, but newcomer might just say "screw it" and go learn React 🙂

@Conduitry Conduitry transferred this issue from sveltejs/svelte Jun 4, 2019
@lukeed
Copy link
Member

lukeed commented Jun 12, 2019

Solved via 3694022 🎉

@lukeed lukeed closed this as completed Jun 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants