diff --git a/CHANGELOG.md b/CHANGELOG.md index 66306d3ce5..62f9af32f6 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,7 @@ This project adheres to [Semantic Versioning](http://semver.org/). ### Fixed - Fixed regression introduced in version 5.0.0 where pandas/numpy arrays with `dtype` of Object were being converted to `list` values when added to a Figure ([#3292](https://github.com/plotly/plotly.py/issues/3292), [#3293](https://github.com/plotly/plotly.py/pull/3293)) + - Better detection of Chrome and Chromium browsers in the Renderers framework, especially on Linux ([#3278](https://github.com/plotly/plotly.py/pull/3278)) with thanks to [@c-chaitanya](https://github.com/c-chaitanya) for the contribution ## [5.1.0] - 2021-06-28 diff --git a/packages/python/plotly/plotly/io/_base_renderers.py b/packages/python/plotly/plotly/io/_base_renderers.py index 66befb8bec..7f1f8e7f25 100644 --- a/packages/python/plotly/plotly/io/_base_renderers.py +++ b/packages/python/plotly/plotly/io/_base_renderers.py @@ -669,15 +669,21 @@ def open_html_in_browser(html, using=None, new=0, autoraise=True): if isinstance(html, six.string_types): html = html.encode("utf8") - if isinstance(using, tuple): - try: - using = [i for i in webbrowser._browsers.keys() if i in using][0] - except IndexError: - raise ValueError( - """ -Unable to find the given browser. -Try one among the following 'chrome', 'chromium', 'firefox' or 'default' """ - ) + browser = None + + if using is None: + browser = webbrowser.get() + else: + if not isinstance(using, tuple): + using = (using,) + for browser_key in using: + try: + browser = webbrowser.get(browser_key) + except webbrowser.Error: + pass + + if browser is None: + raise ValueError("Can't locate a browser with key in " + str(using)) class OneShotRequestHandler(BaseHTTPRequestHandler): def do_GET(self): @@ -694,7 +700,7 @@ def log_message(self, format, *args): pass server = HTTPServer(("127.0.0.1", 0), OneShotRequestHandler) - webbrowser.get(using).open( + browser.open( "http://127.0.0.1:%s" % server.server_port, new=new, autoraise=autoraise )