From 553a88d5746d668a80f19e957ac61af99afc50ab Mon Sep 17 00:00:00 2001 From: Ken Sedgwick Date: Mon, 16 Dec 2024 20:40:35 -0800 Subject: [PATCH] android: use more app top margin for android A more refined solution would query the android environment for the system bar height ... Closes: https://github.com/damus-io/notedeck/pull/585 Reviewed-by: William Casarin Signed-off-by: William Casarin --- crates/notedeck_columns/src/app.rs | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/crates/notedeck_columns/src/app.rs b/crates/notedeck_columns/src/app.rs index 656da3d2..e2b083ff 100644 --- a/crates/notedeck_columns/src/app.rs +++ b/crates/notedeck_columns/src/app.rs @@ -512,9 +512,26 @@ fn render_damus_mobile(app: &mut Damus, app_ctx: &mut AppContext<'_>) { }); } +fn margin_top(narrow: bool) -> f32 { + #[cfg(target_os = "android")] + { + // FIXME - query the system bar height and adjust more precisely + let _ = narrow; // suppress compiler warning on android + 40.0 + } + #[cfg(not(target_os = "android"))] + { + if narrow { + 50.0 + } else { + 0.0 + } + } +} + fn main_panel(style: &Style, narrow: bool) -> egui::CentralPanel { let inner_margin = egui::Margin { - top: if narrow { 50.0 } else { 0.0 }, + top: margin_top(narrow), left: 0.0, right: 0.0, bottom: 0.0,