-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[1.0] Automatically open browser when running gatsby develop (like create-react-app) #1531
Comments
Try
|
Ahh! Should've looked that up before posting this. Sorry about that and thanks for the quick response! |
Opening the site by default is an interesting default. @prayasht do you really like this? We could change the default. I haven't used CRA much or other systems which open pages by default. Is there other tools that do this? The only downside I see is it grabs the focus of your OS which can be annoying. |
I don't like th default of CRA (you can change it with config though), it's quite annoying and you end up with a lot of tabs. As long as it's possible to configure, I think either default is fine. |
@oscar-b On my machine (macOS + Chrome), if an instance is already open in a tab, it simply refreshes it. I've never had it open new instances every time I run the dev server. Honestly, I like it and I just changed my npm script to |
Let's leave this open for a while to see if anyone else has strong feelings about it |
Just chiming in as a create-react-app user to say that I really like the app to open as a default; |
I share the same dislike for Gatsby taking control of my OS windows, even just to launch a browser. I changed the default behavior in CRA the moment I installed it. This can be easily scripted. Most sites already wrap |
Going to chime in and say that |
I too agree, |
Leaving it not as the default :-) |
Is it possible to implement this feature? I find it very handy when starting up the dev server.
Looking through create-react-app
scripts/start.js
, it seems that they use an externalopenBrowser
package like so:If it's a trivial change, I'd love to take a stab at it!
The text was updated successfully, but these errors were encountered: