Use a more loose time for nudging triggers

4 files changed, 36 insertions, 54 deletions

diff --git a/src/drawing.c b/src/drawing.c
index 1d91d86..befbdde 100644
--- a/src/drawing.c
+++ b/src/drawing.c
@@ -1055,11 +1055,14 @@ draw_parameters_wave(ChannelWaveParams *params, bool global) {
         u8 col_env = cols[7] != COL_OFF && cols[8] != COL_OFF ? COL_ACC_2 : COL_OFF;
         if (params->wave_attack != 0) {
             draw_line(x0, y0, x1, y1, col_env);
+        } else {
+            draw_line(x0, y0, x0, y0 - 15, col_env);
         }
         if (params->wave_decay == 0) {
             x2 = x + PARAMS_BOX_OFFSET_X * 2 - 4;
             y2 = y;
             draw_line(x1, y1, x2, y2, col_env);
+            draw_line(x2, y2, x2, y2 + 15, col_env);
         } else {
             draw_line(x1, y1, x2, y2, col_env);
             draw_line(x2, y2, x + PARAMS_BOX_OFFSET_X * 2 - 4, y2, col_env);
@@ -2049,7 +2052,7 @@ draw_notif_bar() {
 
         if (settings.help == TOGGLE_ON) {
             if (input_handler == handle_trigger_selection) {
-                if (key_pressed(KEY_B) && ctrl.key_b > 10) {
+                if (nudge) {
                     txt_draws_small("D-PAD:NUDGE TRIGGER", x0 + 2, y0 + 1, color);
                 } else if (key_pressed(KEY_SELECT)) {
                     txt_draws_small("SEL+B:COPY TRIG SEL+A:PASTE TRIG", x0 + 2, y0 + 1, color);
diff --git a/src/gba/gba.h b/src/gba/gba.h
index 2cc167f..3f629fd 100644
--- a/src/gba/gba.h
+++ b/src/gba/gba.h
@@ -347,64 +347,44 @@ _key_retrig(int key, int offset, int frames) {
     }
     switch (key) {
         case KEY_L: {
-            if (key_hold(key)) {
-                if (ctrl.key_l < offset) { return false; }
-                if (ctrl.key_l % frames == 0) { return true; }
-            }
+            if (ctrl.key_l < offset) { return false; }
+            if (ctrl.key_l % frames == 0) { return true; }
         } break;
         case KEY_R: {
-            if (key_hold(key)) {
-                if (ctrl.key_r < offset) { return false; }
-                if (ctrl.key_r % frames == 0) { return true; }
-            }
+            if (ctrl.key_r < offset) { return false; }
+            if (ctrl.key_r % frames == 0) { return true; }
         } break;
         case KEY_A: {
-            if (key_hold(key)) {
-                if (ctrl.key_a < offset) { return false; }
-                if (ctrl.key_a % frames == 0) { return true; }
-            }
+            if (ctrl.key_a < offset) { return false; }
+            if (ctrl.key_a % frames == 0) { return true; }
         } break;
         case KEY_B: {
-            if (key_hold(key)) {
-                if (ctrl.key_b < offset) { return false; }
-                if (ctrl.key_b % frames == 0) { return true; }
-            }
+            if (ctrl.key_b < offset) { return false; }
+            if (ctrl.key_b % frames == 0) { return true; }
         } break;
         case KEY_SELECT: {
-            if (key_hold(key)) {
-                if (ctrl.key_select < offset) { return false; }
-                if (ctrl.key_select % frames == 0) { return true; }
-            }
+            if (ctrl.key_select < offset) { return false; }
+            if (ctrl.key_select % frames == 0) { return true; }
         } break;
         case KEY_START: {
-            if (key_hold(key)) {
-                if (ctrl.key_start < offset) { return false; }
-                if (ctrl.key_start % frames == 0) { return true; }
-            }
+            if (ctrl.key_start < offset) { return false; }
+            if (ctrl.key_start % frames == 0) { return true; }
         } break;
         case KEY_UP: {
-            if (key_hold(key)) {
-                if (ctrl.key_up < offset) { return false; }
-                if (ctrl.key_up % frames == 0) { return true; }
-            }
+            if (ctrl.key_up < offset) { return false; }
+            if (ctrl.key_up % frames == 0) { return true; }
         } break;
         case KEY_DOWN: {
-            if (key_hold(key)) {
-                if (ctrl.key_down < offset) { return false; }
-                if (ctrl.key_down % frames == 0) { return true; }
-            }
+            if (ctrl.key_down < offset) { return false; }
+            if (ctrl.key_down % frames == 0) { return true; }
         } break;
         case KEY_LEFT: {
-            if (key_hold(key)) {
-                if (ctrl.key_left < offset) { return false; }
-                if (ctrl.key_left % frames == 0) { return true; }
-            }
+            if (ctrl.key_left < offset) { return false; }
+            if (ctrl.key_left % frames == 0) { return true; }
         } break;
         case KEY_RIGHT: {
-            if (key_hold(key)) {
-                if (ctrl.key_right < offset) { return false; }
-                if (ctrl.key_right % frames == 0) { return true; }
-            }
+            if (ctrl.key_right < offset) { return false; }
+            if (ctrl.key_right % frames == 0) { return true; }
         } break;
     }
     return false;
@@ -419,16 +399,16 @@ key_retrig(int key) {
 static inline
 void
 update_controller(void) {
-    if (key_pressed(KEY_UP)) { ctrl.key_up++; } else if (key_released(KEY_UP)) { ctrl.key_up = 0; }
-    if (key_pressed(KEY_DOWN)) { ctrl.key_down++; } else if (key_released(KEY_DOWN)) { ctrl.key_down = 0; }
-    if (key_pressed(KEY_LEFT)) { ctrl.key_left++; } else if (key_released(KEY_LEFT)) { ctrl.key_left = 0; }
-    if (key_pressed(KEY_RIGHT)) { ctrl.key_right++; } else if (key_released(KEY_RIGHT)) { ctrl.key_right = 0; }
-    if (key_pressed(KEY_L)) { ctrl.key_l++; } else if (key_released(KEY_L)) { ctrl.key_l = 0; }
-    if (key_pressed(KEY_R)) { ctrl.key_r++; } else if (key_released(KEY_R)) { ctrl.key_r = 0; }
-    if (key_pressed(KEY_A)) { ctrl.key_a++; } else if (key_released(KEY_A)) { ctrl.key_a = 0; }
-    if (key_pressed(KEY_B)) { ctrl.key_b++; } else if (key_released(KEY_B)) { ctrl.key_b = 0; }
-    if (key_pressed(KEY_SELECT)) { ctrl.key_select++; } else if (key_released(KEY_SELECT)) { ctrl.key_select = 0; }
-    if (key_pressed(KEY_START)) { ctrl.key_start++; } else if (key_released(KEY_START)) { ctrl.key_start = 0; }
+    if (key_pressed(KEY_UP)) { ctrl.key_up++; } else { ctrl.key_up = 0; }
+    if (key_pressed(KEY_DOWN)) { ctrl.key_down++; } else { ctrl.key_down = 0; }
+    if (key_pressed(KEY_LEFT)) { ctrl.key_left++; } else  { ctrl.key_left = 0; }
+    if (key_pressed(KEY_RIGHT)) { ctrl.key_right++; } else  { ctrl.key_right = 0; }
+    if (key_pressed(KEY_L)) { ctrl.key_l++; } else { ctrl.key_l = 0; }
+    if (key_pressed(KEY_R)) { ctrl.key_r++; } else { ctrl.key_r = 0; }
+    if (key_pressed(KEY_A)) { ctrl.key_a++; } else { ctrl.key_a = 0; }
+    if (key_pressed(KEY_B)) { ctrl.key_b++; } else { ctrl.key_b = 0; }
+    if (key_pressed(KEY_SELECT)) { ctrl.key_select++; } else { ctrl.key_select = 0; }
+    if (key_pressed(KEY_START)) { ctrl.key_start++; } else { ctrl.key_start = 0; }
 }
 
 static inline
diff --git a/src/globals.c b/src/globals.c
index 9d0a5bb..b653eaf 100644
--- a/src/globals.c
+++ b/src/globals.c
@@ -19,6 +19,7 @@ u32 last_pattern_loc = 0;
 u32 last_right_col_loc = 0;
 u32 last_step_counter = 0;
 bool clear_screen = true;
+bool nudge = false;
 
 //
 // Color indexes.
diff --git a/src/sequencer.c b/src/sequencer.c
index 4b7d174..9efd506 100644
--- a/src/sequencer.c
+++ b/src/sequencer.c
@@ -951,7 +951,6 @@ handle_pattern_chain(void) {
         switch (param_selection_loc) {
             case CHAIN_BTN_ENABLE: {
                 chain.enabled ^= 1;
-//                chain.playing ^= 1;
             } break;
             case CHAIN_BTN_CLEAR: {
                 chain.len = 0;
@@ -1689,9 +1688,8 @@ void
 handle_trigger_selection(void) {
     TriggerNote *trig = get_current_trig();
 
-    static bool nudge = false;
     bool empty = patterns[pattern_selection_loc].empty;
-    if (key_pressed(KEY_B) && ctrl.key_b > 10) {
+    if (ctrl.key_b > 30) {
         nudge = true;
     } else if (key_released(KEY_B)) {
         if (nudge) {