Add banner about mobile browsers being unsupported (#1674)

* Add banner about missing support on mobile browsers * Clarify mobile isn't supported in general (not just limited to web)
rerun-io · Mar 22, 2023 · f20c7ca · f20c7ca · github-actions · Mar 22, 2023
1 parent 5e660cf
commit f20c7ca
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/crates/re_viewer/src/app.rs b/crates/re_viewer/src/app.rs
@@ -1012,10 +1012,12 @@ impl AppState {
     }
 }
 
-fn warning_panel(re_ui: &re_ui::ReUi, ui: &mut egui::Ui, frame: &mut eframe::Frame) {
-    // We have some bugs when running the web viewer on Windows.
-    // See https://github.com/rerun-io/rerun/issues/1206
-    if frame.is_web() && ui.ctx().os() == egui::os::OperatingSystem::Windows {
+fn warning_panel(re_ui: &re_ui::ReUi, ui: &mut egui::Ui, _frame: &mut eframe::Frame) {
+    // We have not yet optimized the UI experience for mobile. Show a warning banner
+    // with a link to the tracking issue.
+    if ui.ctx().os() == egui::os::OperatingSystem::IOS
+        || ui.ctx().os() == egui::os::OperatingSystem::Android
+    {
         let frame = egui::Frame {
             fill: ui.visuals().panel_fill,
             ..re_ui.bottom_panel_frame()
@@ -1026,10 +1028,9 @@ fn warning_panel(re_ui: &re_ui::ReUi, ui: &mut egui::Ui, frame: &mut eframe::Fra
             .frame(frame)
             .show_inside(ui, |ui| {
                 ui.centered_and_justified(|ui| {
-                    let text = re_ui.warning_text(
-                        "The web viewer has some known issues on Windows. Click for details.",
-                    );
-                    ui.hyperlink_to(text, "https://github.com/rerun-io/rerun/issues/1206");
+                    let text =
+                        re_ui.warning_text("Mobile OSes are not yet supported. Click for details.");
+                    ui.hyperlink_to(text, "https://github.com/rerun-io/rerun/issues/1672");
                 });
             });
     }