plot.show() fix (#129)

* plot.show() fix

Moved a few things around I got this to work on my Windows 10 machine
seems like Spawn() does the trick!!

* Revert "plot.show() fix"

This reverts commit 780976a.

* fix plot.show() Windows

Hi, sorry for the last PR updated!!

* fix command line args

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Michael Freeborn <[email protected]>

Author: juanespj

CHANGELOG.md

@@ -3,6 +3,10 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.8.4] - 2023-01-xx
+### Fixed
+- [[#129](https://github.com/igiagkiozis/plotly/pull/129)] Fix issue for plots not showing in browser in Windows. Thanks to [@juanespj](https://github.com/juanespj) for the PR.
+
 ## [0.8.3] - 2022-11-04
 ### Fixed
 - [[#122](https://github.com/igiagkiozis/plotly/pull/122)] Compilation error for the `wasm` feature.

plotly/src/plot.rs

@@ -466,8 +466,8 @@ impl Plot {
         use std::process::Command;
         Command::new("cmd")
             .arg("/C")
-            .arg(format!(r#"start {}"#, temp_path))
-            .output()
+            .arg(format!("start {}", temp_path))
+            .spawn()
             .expect(DEFAULT_HTML_APP_NOT_FOUND);
     }
 }