some hints on what to implement next

docs/src/loopstation.md

@@ -7,6 +7,15 @@ Hold it down for a while and release it to set the length of the loop.
 
 Notes played while the record button is held will be looped.
 
-The top-right button below the encoder clears the recorded loop.
+The top-right button below the encoder clears the recorded loop in different ways:
+
+ * Pressing it on its own will clear all channels of the current loop, keeping the set length.
+ * Presssing it while holding down one of the channel mute buttons will clear only this channel in the current loop.
+ * Pressing it while holding down the left REC button will clear the current loop, including the length.
+ * Pressing it while holding down the right REC button will clear all loops in all banks.
 
 You can modify the loop length using the encoder menu.
+
+The encoder menu also provides a way to switch between different loop banks.
+With the `Sel-Bnk` menu you can switch to another loop bank.
+With the `Cpy-Bnk` menu you can copy the current loop to another bank and switch to it.

src/sequence.c

@@ -119,30 +119,42 @@ static bool sequence_get(uint32_t beat, enum channels ch) {
 
 void sequence_handle_button_loopstation(enum buttons btn, bool rec) {
     switch (btn) {
-        case BTN_A:
-        case BTN_E: {
+        case BTN_A: {
             pulse_trigger_out(0, mem_data()->ch_timings[0]);
             pulse_trigger_led(0, mem_data()->ch_timings[0]);
             pulse_trigger_led(4, mem_data()->ch_timings[0]);
             break;
         }
 
-        case BTN_B:
-        case BTN_F: {
+        case BTN_B: {
             pulse_trigger_out(1, mem_data()->ch_timings[1]);
             pulse_trigger_led(1, mem_data()->ch_timings[1]);
             pulse_trigger_led(5, mem_data()->ch_timings[1]);
             break;
         }
 
-        case BTN_C:
-        case BTN_G: {
+        case BTN_C: {
             pulse_trigger_out(2, mem_data()->ch_timings[2]);
             pulse_trigger_led(2, mem_data()->ch_timings[2]);
             pulse_trigger_led(6, mem_data()->ch_timings[2]);
             break;
         }
 
+        case BTN_E: {
+            // TODO mute/unmute according to rec param
+            return; // not recording sequence!
+        }
+
+        case BTN_F: {
+            // TODO mute/unmute according to rec param
+            return; // not recording sequence!
+        }
+
+        case BTN_G: {
+            // TODO mute/unmute according to rec param
+            return; // not recording sequence!
+        }
+
         default: {
             break;
         }

src/ui.c

@@ -188,16 +188,20 @@ static void ui_buttons_loopstation(enum buttons btn, bool val) {
     switch (btn) {
         case BTN_A:
         case BTN_B:
-        case BTN_C:
-        case BTN_E:
-        case BTN_F:
-        case BTN_G: {
+        case BTN_C: {
             if (val) {
                 sequence_handle_button_loopstation(btn, rec_held_down);
             }
             break;
         }
 
+        case BTN_E:
+        case BTN_F:
+        case BTN_G: {
+            sequence_handle_button_loopstation(btn, val);
+            break;
+        }
+
         case BTN_REC: {
             // reset sequence
             sequence_init();