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.

Sign up for GitHub

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

Allow --open option to specify the browser to use #825

Merged
merged 18 commits into from
Aug 30, 2017
Merged

Allow --open option to specify the browser to use #825

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
30c4fbd
Add optionally browser parameter to --open
frankwinter Mar 3, 2017
b5f064e
Fix code style issues
frankwinter Mar 3, 2017
5b7f8e0
Fix last code style issue
frankwinter Mar 3, 2017
15ec8d3
Merge remote-tracking branch 'upstream/master' into open-specific-bro…
frankwinter Mar 11, 2017
75ef717
Replace array spread
frankwinter Mar 14, 2017
d78c1ae
Merge remote-tracking branch 'upstream/master' into open-specific-bro…
frankwinter Mar 14, 2017
52ad1a2
Merge remote-tracking branch 'upstream/master' into open-specific-bro…
frankwinter Mar 15, 2017
f04bba3
Fix conflicts
frankwinter Jul 10, 2017
8b64374
Merge branch 'master' of https://github.com/webpack/webpack-dev-serve…
frankwinter Jul 10, 2017
4d65ef1
Fix whitespace issue
frankwinter Jul 10, 2017
2c32834
Add .idea to gitignore and remove files
frankwinter Jul 10, 2017
682d47e
Fix whitespaces
frankwinter Jul 10, 2017
a18ae9c
Ignore and remove package-lock.json
frankwinter Jul 11, 2017
d3a46ae
refactor argument composition
frankwinter Jul 11, 2017
9ca67c9
Merge branch 'master' into open-specific-browser
frankwinter Aug 27, 2017
cb31c90
Merge branch 'master' into open-specific-browser
shellscape Aug 30, 2017
b847645
removing trailing spaces
shellscape Aug 30, 2017
a230754
improve open option description to match pr
shellscape Aug 30, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions bin/webpack-dev-server.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -307,7 +307,7 @@ function processOptions(wpOpt) {
options.compress = true;

if(argv["open"])
options.open = true;
options.open = (argv["open"] === "") ? true : argv["open"];

// Kind of weird, but ensures prior behavior isn't broken in cases
// that wouldn't throw errors. E.g. both argv.port and options.port
Expand Down Expand Up @@ -418,7 +418,12 @@ function reportReadiness(uri, options) {
if(options.historyApiFallback)
console.log(`404s will fallback to ${colorInfo(useColor, options.historyApiFallback.index || "/index.html")}`);
if(options.open) {
open(uri).catch(function() {
var params = [uri];

if(typeof options.open === "string")
params.push({ app: options.open });

open(...params).catch(function() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Array spread doesn't seem to work in Node v4.

console.log("Unable to open browser. If you are running in a headless environment, please do not use the open flag.");
});
}
Expand Down
9 changes: 8 additions & 1 deletion lib/optionsSchema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,14 @@
},
"open": {
"description": "Let the CLI open your browser with the URL.",
"type": "boolean"
"anyOf": [
{
"type": "string"
},
{
"type": "boolean"
}
]
},
"features": {
"description": "The order of which the features will be triggered.",
Expand Down