Fix GUI layout squishing and small default window size

Author: ekulkisnek

app/Cargo.toml

@@ -36,7 +36,7 @@ tonic = { workspace = true }
 tonic-health = { workspace = true }
 # needs to line up with jsonrpsee tower version...
 tower = { workspace = true }
-tower-http = { workspace = true, features = ["request-id", "trace"] }
+tower-http = { workspace = true, features = ["cors", "request-id", "trace"] }
 tracing = { workspace = true }
 tracing-appender = { workspace = true }
 tracing-subscriber = { workspace = true, features = ["env-filter", "json"] }

app/app.rs

@@ -287,30 +287,29 @@ impl App {
         Ok(())
     }
 
-    pub fn get_new_main_address(
+    pub async fn get_new_main_address_async(
         &self,
     ) -> Result<bitcoin::Address<bitcoin::address::NetworkChecked>, Error> {
         let Some(miner) = self.miner.as_ref() else {
             return Err(Error::NoCusfMainchainWalletClient);
         };
-        let address = self.runtime.block_on({
-            let miner = miner.clone();
-            async move {
-                let mut miner_write = miner.write().await;
-                let cusf_mainchain = &mut miner_write.cusf_mainchain;
-                let mainchain_info = cusf_mainchain.get_chain_info().await?;
-                let cusf_mainchain_wallet =
-                    &mut miner_write.cusf_mainchain_wallet;
-                let res = cusf_mainchain_wallet
-                    .create_new_address()
-                    .await?
-                    .require_network(mainchain_info.network)
-                    .unwrap();
-                drop(miner_write);
-                Result::<_, Error>::Ok(res)
-            }
-        })?;
-        Ok(address)
+        let mut miner_write = miner.write().await;
+        let cusf_mainchain = &mut miner_write.cusf_mainchain;
+        let mainchain_info = cusf_mainchain.get_chain_info().await?;
+        let cusf_mainchain_wallet = &mut miner_write.cusf_mainchain_wallet;
+        let res = cusf_mainchain_wallet
+            .create_new_address()
+            .await?
+            .require_network(mainchain_info.network)
+            .unwrap();
+        drop(miner_write);
+        Ok(res)
+    }
+
+    pub fn get_new_main_address(
+        &self,
+    ) -> Result<bitcoin::Address<bitcoin::address::NetworkChecked>, Error> {
+        self.runtime.block_on(self.get_new_main_address_async())
     }
 
     const EMPTY_BLOCK_BMM_BRIBE: bitcoin::Amount =
@@ -531,7 +530,7 @@ impl App {
         Ok(())
     }
 
-    pub fn deposit(
+    pub async fn deposit_async(
         &self,
         address: Address,
         amount: bitcoin::Amount,
@@ -540,15 +539,23 @@ impl App {
         let Some(miner) = self.miner.as_ref() else {
             return Err(Error::NoCusfMainchainWalletClient);
         };
-        self.runtime.block_on(async {
-            let mut miner_write = miner.write().await;
-            let txid = miner_write
-                .cusf_mainchain_wallet
-                .create_deposit_tx(address, amount.to_sat(), fee.to_sat())
-                .await?;
-            drop(miner_write);
-            Ok(txid)
-        })
+        let mut miner_write = miner.write().await;
+        let txid = miner_write
+            .cusf_mainchain_wallet
+            .create_deposit_tx(address, amount.to_sat(), fee.to_sat())
+            .await?;
+        drop(miner_write);
+        Ok(txid)
+    }
+
+    pub fn deposit(
+        &self,
+        address: Address,
+        amount: bitcoin::Amount,
+        fee: bitcoin::Amount,
+    ) -> Result<bitcoin::Txid, Error> {
+        self.runtime
+            .block_on(self.deposit_async(address, amount, fee))
     }
 }
 

app/gui/coins/tx_builder.rs

@@ -48,7 +48,7 @@ impl TxBuilder {
         ui.separator();
         ui.monospace(format!("Total: {value_in}"));
         ui.separator();
-        egui::Grid::new("utxos").striped(true).show(ui, |ui| {
+        egui::Grid::new("spent_utxos").striped(true).show(ui, |ui| {
             ui.monospace("kind");
             ui.monospace("outpoint");
             ui.monospace("value");
@@ -111,33 +111,29 @@ impl TxBuilder {
         ui: &mut egui::Ui,
     ) -> anyhow::Result<()> {
         egui::ScrollArea::horizontal().show(ui, |ui| {
-            egui::SidePanel::left("spend_utxo")
-                .exact_width(300.)
-                .resizable(false)
-                .show_inside(ui, |ui| {
+            ui.horizontal(|ui| {
+                ui.vertical(|ui| {
+                    ui.set_width(250.0);
                     self.utxo_selector.show(app, ui, &mut self.base_tx);
                 });
-            egui::SidePanel::left("value_in")
-                .exact_width(300.)
-                .resizable(false)
-                .show_inside(ui, |ui| {
+                ui.separator();
+                ui.vertical(|ui| {
+                    ui.set_width(250.0);
                     let () = self.show_value_in(app, ui);
                 });
-            egui::SidePanel::left("value_out")
-                .exact_width(250.)
-                .resizable(false)
-                .show_inside(ui, |ui| {
+                ui.separator();
+                ui.vertical(|ui| {
+                    ui.set_width(250.0);
                     let () = self.show_value_out(ui);
                 });
-            egui::SidePanel::left("create_utxo")
-                .exact_width(450.)
-                .resizable(false)
-                .show_separator_line(false)
-                .show_inside(ui, |ui| {
+                ui.separator();
+                ui.vertical(|ui| {
+                    ui.set_width(450.0);
                     self.utxo_creator.show(app, ui, &mut self.base_tx);
                     ui.separator();
                     self.tx_creator.show(app, ui, &mut self.base_tx).unwrap();
                 });
+            });
         });
         Ok(())
     }