Automatic Browser Open (#27)

* feat: open browser window

wait for sometime allowing server to run
Signed-off-by: Lakshya Singh <[email protected]>

* feat: add flag to disable browser opening

check for disable auto open flag before launching go routine
Signed-off-by: Lakshya Singh <[email protected]>

* refactor: log error instead of returning

golint needs to check error if any
Signed-off-by: Lakshya Singh <[email protected]>

Author: king-11

cli.go

@@ -24,6 +24,7 @@ type CLI struct {
 type CLIOptions struct {
 	Port              int    `short:"p" long:"port" description:"TCP port number of this server" default:"3000"`
 	Theme             string `long:"theme" description:"Slide theme or original css file name. default themes: beige, black, blood, league, moon, night, serif, simple, sky, solarized, and white" default:"black.css"`
+	DisableAutoOpen   bool   `long:"disable-auto-open" description:"Disable automatic opening of the browser"`
 	Transition        string `long:"transition" description:"Transition effect for slides: default, cube, page, concave, zoom, linear, fade, none" default:"default"`
 	Separator         string `long:"separator" description:"Horizontal slide separator characters" default:"^---"`
 	VerticalSeparator string `long:"vertical-separator" description:"Vertical slide separator characters" default:"^___"`
@@ -86,6 +87,7 @@ func (cli *CLI) serve(args []string) {
 		Theme:             addExtention(opts.Theme, "css"),
 		Transition:        opts.Transition,
 		OriginalTheme:     originalTheme,
+		DisableAutoOpen:   opts.DisableAutoOpen,
 		Separator:         opts.Separator,
 		VerticalSeparator: opts.VerticalSeparator,
 	}

server.go

@@ -33,6 +33,7 @@ type ServerParam struct {
 	Path              string
 	Theme             string
 	OriginalTheme     bool
+	DisableAutoOpen   bool
 	Transition        string
 	Separator         string
 	VerticalSeparator string
@@ -76,6 +77,9 @@ func (server *Server) Serve(param ServerParam) {
 		fmt.Printf("master slides can be found at http://*:%d/master/\n", port)
 	}
 
+	if !param.DisableAutoOpen {
+		go openBrowser(port)
+	}
 	if err := http.ListenAndServe(fmt.Sprintf(":%d", port), nil); err != nil {
 		log.Fatal("ListenAndServe:", err)
 	}

util.go

@@ -2,9 +2,32 @@ package revealgo
 
 import (
 	"fmt"
+	"os/exec"
+	"runtime"
 	"strings"
+	"time"
 )
 
+func openBrowser(port int) {
+	<-time.After(100 * time.Millisecond)
+	url := fmt.Sprintf("http://localhost:%d/", port)
+	var args []string
+	var cmd string
+	switch runtime.GOOS {
+	case "windows":
+		cmd = "cmd"
+		args = []string{"/c", "start"}
+	case "darwin":
+		cmd = "open"
+	default: // "linux", "freebsd", "openbsd", "netbsd"
+		cmd = "xdg-open"
+	}
+	args = append(args, url)
+	if err := exec.Command(cmd, args...).Start(); err != nil {
+		fmt.Printf("error when trying to open browser: %s", err.Error())
+	}
+}
+
 func addExtention(path string, ext string) string {
 	if strings.HasSuffix(path, fmt.Sprintf(".%s", ext)) {
 		return path