rich text: add an open-url function (#9936)

* Add an open-url function

* Add cpp implementation

* Add workspace dep

* Add core::open_url function

* Remove pub use webbrowser

* Put open_url behind a flag

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: expenses

api/cpp/lib.rs

@@ -230,3 +230,8 @@ pub extern "C" fn slint_set_xdg_app_id(_app_id: &SharedString) {
 pub extern "C" fn slint_detect_operating_system() -> OperatingSystemType {
     i_slint_core::detect_operating_system()
 }
+
+#[unsafe(no_mangle)]
+pub extern "C" fn open_url(url: &SharedString) {
+    i_slint_core::open_url(url)
+}

internal/compiler/expression_tree.rs

@@ -111,6 +111,7 @@ pub enum BuiltinFunction {
     StartTimer,
     StopTimer,
     RestartTimer,
+    OpenUrl,
 }
 
 #[derive(Debug, Clone)]
@@ -276,6 +277,7 @@ declare_builtin_function_types!(
     StartTimer: (Type::ElementReference) -> Type::Void,
     StopTimer: (Type::ElementReference) -> Type::Void,
     RestartTimer: (Type::ElementReference) -> Type::Void,
+    OpenUrl: (Type::String) -> Type::Void
 );
 
 impl BuiltinFunction {
@@ -371,6 +373,7 @@ impl BuiltinFunction {
             BuiltinFunction::StartTimer => false,
             BuiltinFunction::StopTimer => false,
             BuiltinFunction::RestartTimer => false,
+            BuiltinFunction::OpenUrl => false,
         }
     }
 
@@ -448,6 +451,7 @@ impl BuiltinFunction {
             BuiltinFunction::StartTimer => false,
             BuiltinFunction::StopTimer => false,
             BuiltinFunction::RestartTimer => false,
+            BuiltinFunction::OpenUrl => false,
         }
     }
 }

internal/compiler/generator/cpp.rs

@@ -4198,6 +4198,10 @@ fn compile_builtin_function_call(
                 panic!("internal error: invalid args to RetartTimer {arguments:?}")
             }
         }
+        BuiltinFunction::OpenUrl => {
+            let url = a.next().unwrap();
+            format!("slint::cbindgen_private::open_url({})", url)
+        }
     }
 }
 

internal/compiler/generator/rust.rs

@@ -3448,6 +3448,10 @@ fn compile_builtin_function_call(
                 panic!("internal error: invalid args to RestartTimer {arguments:?}")
             }
         }
+        BuiltinFunction::OpenUrl => {
+            let url = a.next().unwrap();
+            quote!(sp::open_url(&#url))
+        }
     }
 }
 

internal/compiler/llr/optim_passes/inline_expressions.rs

@@ -155,6 +155,7 @@ fn builtin_function_cost(function: &BuiltinFunction) -> isize {
         BuiltinFunction::StartTimer => 10,
         BuiltinFunction::StopTimer => 10,
         BuiltinFunction::RestartTimer => 10,
+        BuiltinFunction::OpenUrl => isize::MAX,
     }
 }
 

internal/compiler/lookup.rs

@@ -809,6 +809,7 @@ impl LookupObject for SlintInternal {
         .or_else(|| f("date-now", b(BuiltinFunction::DateNow)))
         .or_else(|| f("valid-date", b(BuiltinFunction::ValidDate)))
         .or_else(|| f("parse-date", b(BuiltinFunction::ParseDate)))
+        .or_else(|| f("open-url", b(BuiltinFunction::OpenUrl)))
     }
 }
 

internal/core/Cargo.toml

@@ -41,6 +41,7 @@ std = [
   "chrono/wasmbind",
   "chrono/clock",
   "dep:sys-locale",
+  "dep:webbrowser",
 ]
 # Unsafe feature meaning that there is only one core running and all thread_local are static.
 # You can only enable this feature if you are sure that any API of this crate is only called
@@ -129,6 +130,8 @@ wgpu-27 = { workspace = true, optional = true }
 
 tr = { workspace = true, optional = true }
 
+webbrowser = { version = "1.0.6", optional = true }
+
 [target.'cfg(target_family = "unix")'.dependencies]
 gettext-rs = { version = "0.7.1", optional = true, features = ["gettext-system"] }
 

internal/core/lib.rs

@@ -148,3 +148,11 @@ pub fn detect_operating_system() -> OperatingSystemType {
 pub fn is_apple_platform() -> bool {
     matches!(detect_operating_system(), OperatingSystemType::Macos | OperatingSystemType::Ios)
 }
+
+#[cfg_attr(not(feature = "std"), allow(unused))]
+pub fn open_url(url: &str) {
+    #[cfg(feature = "std")]
+    if let Err(err) = webbrowser::open(url) {
+        debug_log!("Error opening url {}: {}", url, err);
+    }
+}

internal/interpreter/eval.rs

@@ -1389,6 +1389,12 @@ fn call_builtin_function(
                 panic!("internal error: argument to RestartTimer must be an element")
             }
         }
+        BuiltinFunction::OpenUrl => {
+            let url: SharedString =
+                eval_expression(&arguments[0], local_context).try_into().unwrap();
+            corelib::open_url(&url);
+            Value::Void
+        }
     }
 }