Add FolderDownload icon and implement download directory selection in settings

- Introduced a new FolderDownload icon component for UI.
- Enhanced settings page to allow users to select and display a custom download directory.
- Implemented Tauri commands to get and set the download directory, improving file management capabilities.

Signed-off-by: Pushkar Mishra <pushkarmishra029@gmail.com>

Author: pushkarm029

src-tauri/src/lib.rs

@@ -13,11 +13,16 @@ use tauri::{
 use tokio::sync::mpsc;
 use tokio::sync::Mutex;
 
+/// Application state shared across all Tauri commands.
 struct AppState {
+    /// Iroh instance for peer-to-peer file transfers
     pub iroh: IrohInstance,
+    /// Channel sender for internal event communication
     inner: Mutex<mpsc::Sender<Event>>,
-    // Store active send bubble to keep it alive
+    /// Active send bubble to keep it alive during transfers
     active_send_bubble: Arc<Mutex<Option<SendFilesBubble>>>,
+    /// Custom download directory set by user
+    custom_download_dir: Mutex<Option<PathBuf>>,
 }
 
 enum Event {
@@ -30,6 +35,7 @@ impl AppState {
             iroh,
             inner: Mutex::new(async_proc_input_tx),
             active_send_bubble: Arc::new(Mutex::new(None)),
+            custom_download_dir: Mutex::new(None),
         }
     }
 }
@@ -84,6 +90,8 @@ pub fn run() {
         .invoke_handler(generate_handler![
             generate_ticket,
             receive_files,
+            set_download_directory,
+            get_download_directory,
             open_directory,
             is_valid_ticket,
             get_env
@@ -112,11 +120,28 @@ fn event_handler(message: Event, manager: &AppHandle) {
     }
 }
 
+/// Gets an environment variable value.
+///
+/// # Arguments
+/// * `key` - The environment variable name
+///
+/// # Returns
+/// The value of the environment variable, or an empty string if not found
 #[tauri::command]
 fn get_env(key: &str) -> String {
     std::env::var(String::from(key)).unwrap_or(String::from(""))
 }
 
+/// Generates a ticket for sending files.
+///
+/// Creates a ticket that encodes the file paths and connection information,
+/// which can be shared with a receiver to initiate a file transfer.
+///
+/// # Arguments
+/// * `paths` - List of file paths to include in the transfer
+///
+/// # Returns
+/// A BlobTicket containing the transfer information
 #[tauri::command]
 async fn generate_ticket(
     state: tauri::State<'_, AppState>,
@@ -171,6 +196,16 @@ async fn generate_ticket(
     Ok(ticket)
 }
 
+/// Receives files using a transfer ticket.
+///
+/// Downloads files from a sender using the provided ticket and saves them to
+/// the configured download directory.
+///
+/// # Arguments
+/// * `ticket` - The transfer ticket string from the sender
+///
+/// # Returns
+/// The path to the directory where files were saved
 #[tauri::command]
 async fn receive_files(
     state: tauri::State<'_, AppState>,
@@ -187,12 +222,12 @@ async fn receive_files(
         }
     });
 
-    // Determine output directory
-    let output_dir = if let Some(path) = dirs::download_dir() {
-        path
-    } else {
-        // Android download path
-        PathBuf::from("/storage/emulated/0/Download/")
+    let output_dir = {
+        let custom_dir = state.custom_download_dir.lock().await;
+        custom_dir
+            .clone()
+            .or_else(|| dirs::download_dir())
+            .unwrap_or_else(|| PathBuf::from("/storage/emulated/0/Download/"))
     };
 
     // Receive files with proper file writing
@@ -206,14 +241,80 @@ async fn receive_files(
     Ok(output_dir)
 }
 
+/// Sets a custom download directory for received files.
+///
+/// # Arguments
+/// * `path` - The filesystem path to the directory
+///
+/// # Errors
+/// Returns an error if the path doesn't exist or is not a directory
+#[tauri::command]
+async fn set_download_directory(
+    state: tauri::State<'_, AppState>,
+    path: String,
+) -> Result<(), InvokeError> {
+    let path_buf = PathBuf::from(&path);
+
+    if !path_buf.exists() {
+        return Err(InvokeError::from_anyhow(anyhow!(
+            "Directory does not exist: {}",
+            path
+        )));
+    }
+
+    if !path_buf.is_dir() {
+        return Err(InvokeError::from_anyhow(anyhow!(
+            "Path is not a directory: {}",
+            path
+        )));
+    }
+
+    let mut custom_dir = state.custom_download_dir.lock().await;
+    *custom_dir = Some(path_buf);
+
+    Ok(())
+}
+
+/// Gets the current download directory.
+///
+/// Returns the custom directory if set, otherwise returns the system default
+/// download directory, or the Android download path as a fallback.
+///
+/// # Returns
+/// The absolute path to the download directory as a string
+#[tauri::command]
+async fn get_download_directory(state: tauri::State<'_, AppState>) -> Result<String, InvokeError> {
+    let custom_dir = state.custom_download_dir.lock().await;
+
+    let output_dir = custom_dir
+        .clone()
+        .or_else(|| dirs::download_dir())
+        .unwrap_or_else(|| PathBuf::from("/storage/emulated/0/Download/"));
+
+    Ok(output_dir.to_string_lossy().to_string())
+}
+
+/// Opens a directory in the system's file manager.
+///
+/// # Arguments
+/// * `directory` - Path to the directory to open
+///
+/// # Errors
+/// Returns an error if the directory cannot be opened
 #[tauri::command]
 fn open_directory(directory: PathBuf) -> Result<(), InvokeError> {
     open::that(directory).map_err(|e| InvokeError::from_anyhow(anyhow!(e)))
 }
 
+/// Validates a transfer ticket format.
+///
+/// # Arguments
+/// * `ticket` - The ticket string to validate
+///
+/// # Returns
+/// `true` if the ticket is valid, `false` otherwise
 #[tauri::command]
 fn is_valid_ticket(ticket: String) -> Result<bool, InvokeError> {
-    // With ark-core, we can simply try to parse the ticket
     match BlobTicket::parse(&ticket) {
         Ok(_) => Ok(true),
         Err(_) => Ok(false),

src/lib/components/icons/FolderDownload.svelte

@@ -0,0 +1,15 @@
+<svg
+	{...$$props}
+	width="100%"
+	height="100%"
+	viewBox="0 0 24 24"
+	fill="none"
+	xmlns="http://www.w3.org/2000/svg"
+>
+	<path
+		d="M13 10L12 11M12 11L11 10M12 11V7M22 11V17.2C22 18.8802 22 19.7202 21.673 20.362C21.3854 20.9265 20.9265 21.3854 20.362 21.673C19.7202 22 18.8802 22 17.2 22H6.8C5.11984 22 4.27976 22 3.63803 21.673C3.07354 21.3854 2.6146 20.9265 2.32698 20.362C2 19.7202 2 18.8802 2 17.2V6.8C2 5.11984 2 4.27976 2.32698 3.63803C2.6146 3.07354 3.07354 2.6146 3.63803 2.32698C4.27976 2 5.11984 2 6.8 2H8.34315C9.16065 2 9.5694 2 9.93694 2.15224C10.2619 2.28624 10.5497 2.49827 10.7764 2.77091C11.0337 3.07908 11.1816 3.49013 11.4775 4.31224L12.5225 7.18776C12.8184 8.00987 12.9663 8.42092 13.2236 8.72909C13.4503 9.00173 13.7381 9.21376 14.0631 9.34776C14.4306 9.5 14.8394 9.5 15.6569 9.5H17.2C18.8802 9.5 19.7202 9.5 20.362 9.82698C20.9265 10.1146 21.3854 10.5735 21.673 11.138C21.8657 11.5 21.9523 11.9561 21.9827 12.5"
+		stroke-width="2"
+		stroke-linecap="round"
+		stroke-linejoin="round"
+	/>
+</svg>

src/routes/settings/+page.svelte

@@ -2,10 +2,41 @@
 	import NavBar from '$lib/components/NavBar.svelte';
 	import Edit05 from '$lib/components/icons/Edit05.svelte';
 	import File06 from '$lib/components/icons/File06.svelte';
+	import FolderDownload from '$lib/components/icons/FolderDownload.svelte';
 	import ShieldTick from '$lib/components/icons/ShieldTick.svelte';
 	import MessageQuestionSquare from '$lib/components/icons/MessageQuestionSquare.svelte';
 	import Star01 from '$lib/components/icons/Star01.svelte';
 	import { goto } from '$app/navigation';
+	import { onMount } from 'svelte';
+	import { invoke } from '@tauri-apps/api/core';
+	import { open } from '@tauri-apps/plugin-dialog';
+
+	let currentDirectory = '';
+
+	onMount(async () => {
+		try {
+			currentDirectory = await invoke('get_download_directory');
+		} catch (error) {
+			console.error('Failed to get download directory:', error);
+		}
+	});
+
+	async function selectDownloadDirectory() {
+		try {
+			const selected = await open({
+				directory: true,
+				multiple: false,
+				title: 'Select Download Directory'
+			});
+
+			if (selected && typeof selected === 'string') {
+				await invoke('set_download_directory', { path: selected });
+				currentDirectory = selected;
+			}
+		} catch (error) {
+			console.error('Failed to set download directory:', error);
+		}
+	}
 </script>
 
 <div class="flex w-full flex-col bg-blue-dark-500 p-4">
@@ -32,7 +63,21 @@
 <ul
 	class="my-3 flex flex-col gap-3 stroke-nav-item-icon-fg p-4 font-semibold text-text-secondary-700"
 >
-	<li class="flex flex-row gap-3 px-3 py-2"><File06 class=" h-6 w-6" />Tems of service</li>
+	<li>
+		<button
+			on:click={selectDownloadDirectory}
+			class="flex w-full flex-row items-center gap-3 rounded-lg px-3 py-2 hover:bg-gray-modern-100"
+		>
+			<FolderDownload class="h-6 w-6 stroke-nav-item-icon-fg" />
+			<div class="flex flex-1 flex-col items-start">
+				<span>Download Directory</span>
+				{#if currentDirectory}
+					<span class="text-xs font-normal text-gray-modern-500">{currentDirectory}</span>
+				{/if}
+			</div>
+		</button>
+	</li>
+	<li class="flex flex-row gap-3 px-3 py-2"><File06 class=" h-6 w-6" />Terms of service</li>
 	<li class="flex flex-row gap-3 px-3 py-2">
 		<ShieldTick class="h-6 w-6 stroke-nav-item-icon-fg" />Privacy Policy
 	</li>