fix(core): set dialog title via API, closes #4029 (#4030)

lucasfernog · web-flow · commit 220e7460148d · 2022-05-02T21:04:45.000-03:00
diff --git a/.changes/fix-dialog-title.md b/.changes/fix-dialog-title.md
@@ -0,0 +1,5 @@
+---
+"tauri": patch
+---
+
+Fixes the `title` option being ignored in the dialog API endpoints.
diff --git a/core/tauri/src/endpoints/dialog.rs b/core/tauri/src/endpoints/dialog.rs
@@ -95,6 +95,9 @@ impl Cmd {
     {
       dialog_builder = dialog_builder.set_parent(&context.window);
     }
+    if let Some(title) = options.title {
+      dialog_builder = dialog_builder.set_title(&title);
+    }
     if let Some(default_path) = options.default_path {
       dialog_builder = set_default_path(dialog_builder, default_path);
     }
@@ -143,6 +146,9 @@ impl Cmd {
     {
       dialog_builder = dialog_builder.set_parent(&context.window);
     }
+    if let Some(title) = options.title {
+      dialog_builder = dialog_builder.set_title(&title);
+    }
     if let Some(default_path) = options.default_path {
       dialog_builder = set_default_path(dialog_builder, default_path);
     }

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,9 @@ impl Cmd {`
`95`	`95`	`{`
`96`	`96`	`dialog_builder = dialog_builder.set_parent(&context.window);`
`97`	`97`	`}`
	`98`	`+ if let Some(title) = options.title {`
	`99`	`+ dialog_builder = dialog_builder.set_title(&title);`
	`100`	`+ }`
`98`	`101`	`if let Some(default_path) = options.default_path {`
`99`	`102`	`dialog_builder = set_default_path(dialog_builder, default_path);`
`100`	`103`	`}`
`@@ -143,6 +146,9 @@ impl Cmd {`
`143`	`146`	`{`
`144`	`147`	`dialog_builder = dialog_builder.set_parent(&context.window);`
`145`	`148`	`}`
	`149`	`+ if let Some(title) = options.title {`
	`150`	`+ dialog_builder = dialog_builder.set_title(&title);`
	`151`	`+ }`
`146`	`152`	`if let Some(default_path) = options.default_path {`
`147`	`153`	`dialog_builder = set_default_path(dialog_builder, default_path);`
`148`	`154`	`}`