(#1221) improve UX of clicking on level in level picker

diff --git a/src/game/level_picker.c b/src/game/level_picker.c
index 4066fe602020b84d7cc6f37fe29795befd27427e..5fafb9824bc6b2df60127638efe47f27f7fd45ac 100644 (file)
--- a/src/game/level_picker.c
+++ b/src/game/level_picker.c
@@ -254,31 +254,35 @@ int level_picker_event(LevelPicker *level_picker,
         }
     } break;
 
-    case SDL_MOUSEMOTION: {
-        const Vec2f mouse_pos = vec((float) event->motion.x, (float) event->motion.y);
-        Vec2f position = vec_sum(
-            level_picker->items_position,
-            level_picker->items_scroll);
+    case SDL_MOUSEBUTTONDOWN: {
+        switch (event->button.button) {
+        case SDL_BUTTON_LEFT: {
+            const Vec2f mouse_pos = vec((float) event->button.x, (float) event->button.y);
+            Vec2f position = vec_sum(
+                level_picker->items_position,
+                level_picker->items_scroll);
 
-        for (size_t i = 0; i < level_picker->items.count; ++i) {
-            const char *item_text = dynarray_pointer_at(
-                &level_picker->items,
-                i);
+            for (size_t i = 0; i < level_picker->items.count; ++i) {
+                const char *item_text = dynarray_pointer_at(
+                    &level_picker->items,
+                    i);
 
-            Rect boundary_box = sprite_font_boundary_box(
-                position,
-                LEVEL_PICKER_LIST_FONT_SCALE,
-                item_text);
+                Rect boundary_box = sprite_font_boundary_box(
+                    position,
+                    LEVEL_PICKER_LIST_FONT_SCALE,
+                    item_text);
 
-            if (rect_contains_point(boundary_box, mouse_pos)) {
-                level_picker->items_cursor = i;
-            }
+                if (rect_contains_point(boundary_box, mouse_pos)) {
+                    level_picker->items_cursor = i;
+                }
 
-            position.y += boundary_box.h + LEVEL_PICKER_LIST_PADDING_BOTTOM;
+                position.y += boundary_box.h + LEVEL_PICKER_LIST_PADDING_BOTTOM;
+            }
+        } break;
         }
     } break;
 
-    case SDL_MOUSEBUTTONDOWN: {
+    case SDL_MOUSEBUTTONUP: {
         switch (event->button.button) {
         case SDL_BUTTON_LEFT: {
             // check if the click position was actually inside...