From c23764c6efc9b8649f9576f7526bb8feb96b772e Mon Sep 17 00:00:00 2001
From: Victoria Fischer <tdfischer@hackerbots.net>
Date: Mon, 9 Mar 2026 10:01:54 +0100
Subject: [PATCH] tasks: ui: docs++

---
 src/tasks/ui.rs | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/tasks/ui.rs b/src/tasks/ui.rs
index 7df9c1c..9807a1f 100644
--- a/src/tasks/ui.rs
+++ b/src/tasks/ui.rs
@@ -89,7 +89,6 @@ impl<S: Debug + Surface<Uniforms = Uniforms, CoordinateSpace = SegmentSpace, Pix
         self.as_slice().set_visible(true);
     }
 
-    // TODO: Brakelight should only be toggled when actually braking or stationary
     pub async fn apply_scene(&mut self, next_scene: Scene) {
         info!("Activating scene {next_scene:?}");
         match next_scene {
@@ -107,12 +106,10 @@ impl<S: Debug + Surface<Uniforms = Uniforms, CoordinateSpace = SegmentSpace, Pix
                 self.background.set_shader(Background::default());
             },
             Scene::Idle => {
-                // FIXME: The safety UI task should handle setting the display brightness to 50% here
                 self.background.set_shader(Thinking::default());
                 let fg_fade = Animation::default().duration(Duration::from_millis(300)).to(Fract8::MIN);
                 let bg_fade = Animation::default().duration(Duration::from_millis(300)).to(Fract8::from_raw(128));
                 
-                // FIXME: The scenes shouldn't be touching the brake/headlights at all here. In fact, they should be dealt with in a whole separate task from the main UI, maybe running on the motion prediction executor
                 embassy_futures::join::join(
                     fg_fade.apply([&mut self.tail, &mut self.panels, &mut self.motion]),
                     bg_fade.apply([&mut self.background])