diff options
Diffstat (limited to 'src/app/input/mod.rs')
| -rw-r--r-- | src/app/input/mod.rs | 29 |
1 files changed, 27 insertions, 2 deletions
diff --git a/src/app/input/mod.rs b/src/app/input/mod.rs index 45124107..597d9296 100644 --- a/src/app/input/mod.rs +++ b/src/app/input/mod.rs @@ -567,6 +567,15 @@ impl App { source_id: super::InputSourceId, mouse: MouseEvent, ) -> bool { + self.handle_modified_url_click_with(source_id, mouse, crate::platform::open_url) + } + + fn handle_modified_url_click_with( + &mut self, + source_id: super::InputSourceId, + mouse: MouseEvent, + open_url: impl FnOnce(&str) -> std::io::Result<Option<std::process::Child>>, + ) -> bool { if self.state.mode != Mode::Terminal || !matches!(mouse.kind, MouseEventKind::Down(MouseButton::Left)) || !mouse.modifiers.contains(modified_url_click_modifier()) @@ -595,8 +604,12 @@ impl App { tracing::warn!(err = %err, url = %url, "failed to invoke plugin link handler"); } } - if let Err(err) = crate::platform::open_url(&url) { - tracing::warn!(err = %err, url = %url, "failed to open pane URL"); + match open_url(&url) { + Ok(Some(child)) => self.detached_process_children.push(child), + Ok(None) => {} + Err(err) => { + tracing::warn!(err = %err, url = %url, "failed to open pane URL"); + } } true } @@ -898,6 +911,18 @@ fn wait_for_file(path: &std::path::Path) -> String { } #[cfg(test)] +#[cfg(unix)] +async fn wait_for_detached_process_reap(app: &mut App, pid: u32) -> bool { + let deadline = tokio::time::Instant::now() + std::time::Duration::from_secs(2); + while crate::platform::process_exists(pid) && tokio::time::Instant::now() < deadline { + app.reap_finished_detached_processes(); + tokio::time::sleep(std::time::Duration::from_millis(20)).await; + } + app.reap_finished_detached_processes(); + !crate::platform::process_exists(pid) +} + +#[cfg(test)] mod tests { use super::*; |
