Change to separate method

altair/vegalite/v5/api.py

@@ -1104,31 +1104,36 @@ def to_html(
             **kwargs,
         )
 
-    def to_url(self, *, open_browser: bool = True, fullscreen: bool = False) -> str:
+    def to_url(self, *, fullscreen: bool = False) -> str:
         """Convert a chart to a URL that opens the chart specification in the Vega chart editor
         The chart specification (including any inline data) is encoded in the URL.
 
         This method requires that the vl-convert-python package is installed.
 
         Parameters
         ----------
-        open_browser : bool
-            If True, the default browser will be launched to open the url. Default True
         fullscreen : bool
             If True, editor will open chart in fullscreen mode. Default False
         """
         from ...utils._importers import import_vl_convert
 
         vlc = import_vl_convert()
         if _using_vegafusion():
-            url = vlc.vega_to_url(self.to_dict(format="vega"), fullscreen=fullscreen)
+            return vlc.vega_to_url(self.to_dict(format="vega"), fullscreen=fullscreen)
         else:
-            url = vlc.vegalite_to_url(self.to_dict(), fullscreen=fullscreen)
-        if open_browser:
-            import webbrowser
+            return vlc.vegalite_to_url(self.to_dict(), fullscreen=fullscreen)
 
-            webbrowser.open(url)
-        return url
+    def open_editor(self, *, fullscreen: bool = False) -> None:
+        """Opens the chart specification in the Vega chart editor using the default browser.
+
+        Parameters
+        ----------
+        fullscreen : bool
+            If True, editor will open chart in fullscreen mode. Default False
+        """
+        import webbrowser
+
+        webbrowser.open(self.to_url(fullscreen = fullscreen))
 
     def save(
         self,

tests/vegalite/v5/test_api.py

@@ -381,7 +381,7 @@ def test_save_html(basic_chart, inline):
 
 
 def test_to_url(basic_chart):
-    share_url = basic_chart.to_url(open_browser=False)
+    share_url = basic_chart.to_url()
     expected_vegalite_encoding = "N4Igxg9gdgZglgcxALlANzgUwO4tJKAFzigFcJSBnAdTgBNCALFAZgAY2AacaYsiygAlMiRoVYcAvpO50AhoTl4QUOQFtMKEPMUBaAOwA2ABwAWFi1NyTcgEb7TtuabAswc-XTZhMczLdNDAEYQGRA1OQAnAGtlQgBPAAdNZBAnSNDuTChIOhIkVBAAD2V4TAAbOi0lbgTkrSgINRI5csyQeNKsSq1bEFqklJAAR1I5IjhFYjRNaW4AEkowRkwIrTFCRMpkAHodmYQ5ADoEScZSWyO4CB2llYj9zEPdcsnMfYBWI6DDI5YjgBWlGg-W0CjklEwhEoyh0cgMJnMlmsxjsDicLjcHi8Pj8AWCKAA2qAlKkAIKgvrIABMxhkJK0ACFKSgPh96SBSSAAMIs5DmDlcgAifIAnEFBVoAKJ84wSzgM1IAMT5HxYktSAHE+UFRRqQIJZfp9QBJVXUyQAXWkQA"
 
     assert (
@@ -390,7 +390,7 @@ def test_to_url(basic_chart):
     )
 
     # Check fullscreen
-    fullscreen_share_url = basic_chart.to_url(open_browser=False, fullscreen=True)
+    fullscreen_share_url = basic_chart.to_url(fullscreen=True)
     assert (
         fullscreen_share_url
         == f"https://vega.github.io/editor/#/url/vega-lite/{expected_vegalite_encoding}/view"