diff --git a/crates/viewer/re_viewer/src/ui/settings_screen.rs b/crates/viewer/re_viewer/src/ui/settings_screen.rs index e1fcf6564dea..f1c749503ae2 100644 --- a/crates/viewer/re_viewer/src/ui/settings_screen.rs +++ b/crates/viewer/re_viewer/src/ui/settings_screen.rs @@ -123,13 +123,13 @@ fn settings_screen_ui_impl(ui: &mut egui::Ui, app_options: &mut AppOptions, keep { ui.re_checkbox( &mut app_options.video_decoder_override_ffmpeg_path, - "Override ffmpeg path", + "Override the FFmpeg binary path", ) .on_hover_ui(|ui| { ui.markdown_ui( - "By default, the viewer tries to automatically find a suitable `ffmpeg` binary in \ + "By default, the viewer tries to automatically find a suitable FFmpeg binary in \ the system's `PATH`. Enabling this option allows you to specify a custom path to \ - the `ffmpeg` binary.", + the FFmpeg binary.", ); }); @@ -147,7 +147,7 @@ fn settings_screen_ui_impl(ui: &mut egui::Ui, app_options: &mut AppOptions, keep // This affects only the web target, so we dont need to show it on native. #[cfg(target_arch = "wasm32")] { - let hardware_acceleration = &mut app_options.video_decoder_settings.hw_acceleration; + let hardware_acceleration = &mut app_options.video_decoder_hw_acceleration; ui.horizontal(|ui| { ui.label("Decoder:"); egui::ComboBox::from_id_salt("video_decoder_hw_acceleration") diff --git a/crates/viewer/re_viewer_context/src/app_options.rs b/crates/viewer/re_viewer_context/src/app_options.rs index 83cac6855901..5c986aa9816b 100644 --- a/crates/viewer/re_viewer_context/src/app_options.rs +++ b/crates/viewer/re_viewer_context/src/app_options.rs @@ -38,16 +38,16 @@ pub struct AppOptions { /// Preferred method for video decoding on web. pub video_decoder_hw_acceleration: DecodeHardwareAcceleration, - /// Override the path to the ffmpeg binary. + /// Override the path to the FFmpeg binary. /// - /// If set, use `video_decoder_ffmpeg_path` as the path to the ffmpeg binary. + /// If set, use `video_decoder_ffmpeg_path` as the path to the FFmpeg binary. /// Don't use this field directly, use [`AppOptions::video_decoder_settings`] instead. /// /// Implementation note: we avoid using `Option` here to avoid loosing the user-defined /// path when disabling the override. pub video_decoder_override_ffmpeg_path: bool, - /// Custom path to the ffmpeg binary. + /// Custom path to the FFmpeg binary. /// /// Don't use this field directly, use [`AppOptions::video_decoder_settings`] instead. pub video_decoder_ffmpeg_path: String,