Merge branch 'MHP30' of https://github.com/Ralim/IronOS into MHP30

Translations/translation_DE.json

@@ -15,7 +15,7 @@
 		"InputVoltageString": "V Eingang: ",
 		"WarningTipTempString": "Temperatur: ",
 		"BadTipString": "Spitze Defekt",
-		"SleepingSimpleString": "Zzz ",
+		"SleepingSimpleString": "Zzzz",
 		"SleepingAdvancedString": "Ruhemodus...",
 		"WarningSimpleString": "HEISS!",
 		"WarningAdvancedString": "! Achtung Heiß !",
@@ -34,12 +34,12 @@
 			"zurückgesetzt!"
 		],
 		"NoAccelerometerMessage": [
-			"Kein Bewegungssensor",
+			"Bewegungssensor",
-			"erkannt!"
+			"nicht erkannt!"
 		],
 		"NoPowerDeliveryMessage": [
-			"Kein USB-PD IC",
+			"USB-PD IC",
-			"erkannt!"
+			"nicht erkannt!"
 		],
 		"LockingKeysString": "GESPERRT",
 		"UnlockingKeysString": "ENTSPERRT",
@@ -128,7 +128,7 @@
 		"ShutdownTimeout": {
 			"text2": [
 				"Abschalt-",
-				"zeit"
+				"verzög."
 			],
 			"desc": "Dauer vor automatischer Abschaltung <M=Minuten>"
 		},
@@ -219,14 +219,14 @@
 		"QCMaxVoltage": {
 			"text2": [
 				"Spannungs-",
-				"grenze"
+				"maximum"
 			],
 			"desc": "Maximal zulässige Spannung der verwendeten Spannungsversorgung <V=Volt>"
 		},
 		"PowerLimit": {
 			"text2": [
 				"Leistungs-",
-				"grenze"
+				"maximum"
 			],
 			"desc": "Maximale zulässige Leistungsaufnahme des Lötkolbens <W=Watt>"
 		},
@@ -260,8 +260,8 @@
 		},
 		"HallEffSensitivity": {
 			"text2": [
-				"Hall-Sonde",
+				"Empfindlichkeit",
-				"Empfindlichkeit"
+				"der Hall-Sonde"
 			],
 			"desc": "Empfindlichkeit der Hall-Sonde beim Erkennen des Ruhemodus <A=aus | N=niedrig | M=mittel | H=hoch>"
 		},
@@ -274,17 +274,17 @@
 		},
 		"MinVolCell": {
 			"text2": [
-				"Minimum",
+				"Minimale",
 				"Spannung"
 			],
-			"desc": "Minimal zulässige Spannung pro Zelle <Volt> <3S: 3,0V - 3,7V, 4/5/6S: 2,4V - 3,7V>"
+			"desc": "Minimal zulässige Spannung pro Zelle <3S: 3,0V - 3,7V | 4/5/6S: 2,4V - 3,7V>"
 		},
 		"AnimLoop": {
 			"text2": [
 				"Anim.",
 				"Schleife"
 			],
-			"desc": "Icon-Animationen im Stammmenü wiederholen"
+			"desc": "Icon-Animationen im Hauptmenü wiederholen"
 		},
 		"AnimSpeed": {
 			"text2": [
@@ -295,21 +295,21 @@
 		},
 		"PowerPulseWait": {
 			"text2": [
-				"Leistungsimpulse",
+				"Impuls-",
-				"Wartezeit"
+				"verzögerung"
 			],
-			"desc": "Dauer vor Abgabe von Wachhalteimpulsen (x 2,5s)"
+			"desc": "Dauer vor Abgabe von Wachhalteimpulsen <x 2,5s>"
 		},
 		"PowerPulseDuration": {
 			"text2": [
-				"Leistungsimpulse",
+				"Impuls-",
-				"Dauer"
+				"dauer"
 			],
-			"desc": "Dauer des Wachhalteimpulses (x 250ms)"
+			"desc": "Dauer des Wachhalteimpulses <x 0,25s>"
 		},
 		"LanguageSwitch": {
 			"text2": [
-				"Language:",
+				"Sprache:",
 				" DE     Deutsch"
 			],
 			"desc": ""

Translations/translation_IT.json

@@ -218,7 +218,7 @@
 		"QCMaxVoltage": {
 			"text2": [
 				"Voltaggio",
-				"Quick Charge"
+				"QC"
 			],
 			"desc": "Imposta il massimo voltaggio negoziabile con un alimentatore Quick Charge"
 		},
@@ -269,14 +269,14 @@
 				"Blocco",
 				"tasti"
 			],
-			"desc": "Blocca i tasti durante la modalità Saldatura; tieni premuto entrambi per bloccare o sbloccare [D: disattiva; T: blocca Turbo; C: blocco completo]"
+			"desc": "Blocca i tasti durante la modalità Saldatura; tieni premuto entrambi per bloccare o sbloccare [D: disattiva; T: consenti Turbo; C: blocco completo]"
 		},
 		"MinVolCell": {
 			"text2": [
 				"Tensione",
 				"min celle"
 			],
-			"desc": "Modifica il valore di tensione minima \"di scaricamento\" per le celle di una batteria Li-Po [3S: 3,0-3,7 V; 4S/5S/6S: 2,4-3,7 V]"
+			"desc": "Modifica la tensione di minima carica delle celle di una batteria Li-Po [3S: 3,0-3,7 V; 4S/5S/6S: 2,4-3,7 V]"
 		},
 		"AnimLoop": {
 			"text2": [
@@ -308,7 +308,7 @@
 		},
 		"LanguageSwitch": {
 			"text2": [
-				"Language:",
+				"Lingua:",
 				" IT    Italiano"
 			],
 			"desc": ""

source/Core/Src/gui.cpp

@@ -973,6 +973,8 @@ static bool settings_setHallEffect(void) {
 }
 #endif
 
+// Indicates whether a menu transition is in progress, so that the menu icon
+// animation is paused during the transition.
 static bool animOpenState = false;
 
 static void displayMenu(size_t index) {
@@ -984,7 +986,6 @@ static void displayMenu(size_t index) {
   // 2 pixel wide scrolling indicator
   static TickType_t menuSwitchLoopTick = 0;
   static size_t     menuCurrentIndex   = sizeof(rootSettingsMenu) + 1;
-  static size_t     currentFrame       = 0;
   TickType_t        step               = TICKS_100MS * 5;
   switch (systemSettings.animationSpeed) {
   case settingOffSpeed_t::FAST:
@@ -996,17 +997,25 @@ static void displayMenu(size_t index) {
   default: // SLOW or off - defaulted above
     break;
   }
-  if (!animOpenState) {
+  size_t currentFrame;
+  if (!animOpenState && systemSettings.animationSpeed != settingOffSpeed_t::OFF) {
     if (menuCurrentIndex != index) {
       menuCurrentIndex   = index;
-      currentFrame       = systemSettings.animationSpeed == settingOffSpeed_t::OFF ? 2 : 0;
       menuSwitchLoopTick = xTaskGetTickCount();
     }
-    if (systemSettings.animationSpeed && (systemSettings.animationLoop || currentFrame != 2)) {
+    currentFrame = ((xTaskGetTickCount() - menuSwitchLoopTick) / step);
-      currentFrame = ((xTaskGetTickCount() - menuSwitchLoopTick) / step) % 3;
+    if (systemSettings.animationLoop) {
+      currentFrame %= 3;
+    } else if (currentFrame > 2) {
+      currentFrame = 2;
+    }
+  } else {
+    // We want the animation to restart after completing the transition.
+    menuCurrentIndex = sizeof(rootSettingsMenu) + 1;
+    // Always draw the last frame if icon animation is disabled.
+    currentFrame = systemSettings.animationSpeed == settingOffSpeed_t::OFF ? 2 : 0;
   }
   OLED::drawArea(OLED_WIDTH - 16 - 2, 0, 16, 16, (&SettingsMenuIcons[index][(16 * 2) * currentFrame]));
-  }
 }
 
 static bool settings_displayCalibrateVIN(void) {
@@ -1059,6 +1068,18 @@ static bool settings_enterAdvancedMenu(void) {
 
 void gui_Menu(const menuitem *menu) {
   // Draw the settings menu and provide iteration support etc
+
+  // This is used to detect whether a menu-exit transition should be played.
+  static bool wasInGuiMenu;
+  wasInGuiMenu = true;
+
+  enum class NavState {
+    Idle,
+    Entering,
+    ScrollingDown,
+    Exiting,
+  };
+
   uint8_t     currentScreen          = 0;
   TickType_t  autoRepeatTimer        = 0;
   TickType_t  autoRepeatAcceleration = 0;
@@ -1068,7 +1089,7 @@ void gui_Menu(const menuitem *menu) {
   uint8_t     scrollContentSize      = 0;
   bool        scrollBlink            = false;
   bool        lastValue              = false;
-  bool        scrollingDown          = false;
+  NavState    navState               = NavState::Entering;
 
   ScrollMessage scrollMessage;
 
@@ -1076,42 +1097,58 @@ void gui_Menu(const menuitem *menu) {
     scrollContentSize += 1;
   }
 
-  // Animated menu opening.
+  while ((menu[currentScreen].draw != nullptr) && earlyExit == false) {
-  if (menu[currentScreen].draw != nullptr) {
+
-    // This menu is drawn in a secondary framebuffer.
+    // Handle menu transition:
-    // Then we play a transition from the current primary
+    if (navState != NavState::Idle) {
-    // framebuffer to the new buffer.
+      // Check if this menu item shall be skipped. If it shall be skipped,
-    // The extra buffer is discarded at the end of the transition.
+      // `draw()` returns true. Draw on the secondary framebuffer as we want
-    animOpenState = true;
+      // to keep the primary framebuffer intact for the upcoming transition
+      // animation.
       OLED::useSecondaryFramebuffer(true);
+      if (menu[currentScreen].draw()) {
+        currentScreen++;
+        OLED::useSecondaryFramebuffer(false);
+        continue;
+      }
+
+      animOpenState = true;
+      // The menu entering/exiting transition uses the secondary framebuffer,
+      // but the scroll down transition does not.
+      if (navState == NavState::ScrollingDown) {
+        OLED::useSecondaryFramebuffer(false);
+      }
       OLED::setCursor(0, 0);
       OLED::clearScreen();
       menu[currentScreen].draw();
+      if (navState == NavState::ScrollingDown) {
+        // Play the scroll down animation.
+        OLED::maskScrollIndicatorOnOLED();
+        OLED::transitionScrollDown();
+      } else {
+        // The menu was drawn in a secondary framebuffer.
+        // Now we play a transition from the pre-drawn primary
+        // framebuffer to the new buffer.
+        // The extra buffer is discarded at the end of the transition.
         OLED::useSecondaryFramebuffer(false);
-    OLED::transitionSecondaryFramebuffer(true);
+        OLED::transitionSecondaryFramebuffer(navState == NavState::Entering);
-    animOpenState = false;
       }
-
+      animOpenState = false;
-  while ((menu[currentScreen].draw != nullptr) && earlyExit == false) {
+      navState      = NavState::Idle;
-    OLED::setCursor(0, 0);
-    if (scrollingDown) {
-      animOpenState = true;
     }
 
     // If the user has hesitated for >=3 seconds, show the long text
     // Otherwise "draw" the option
     if ((xTaskGetTickCount() - lastButtonTime < (TICKS_SECOND * 3)) || menu[currentScreen].description == 0) {
       lcdRefresh = true;
+      OLED::setCursor(0, 0);
       OLED::clearScreen();
-      if (menu[currentScreen].draw()) {
+      menu[currentScreen].draw();
-        currentScreen++;
-        lcdRefresh = false;
-      }
       uint8_t indicatorHeight = OLED_HEIGHT / scrollContentSize;
       uint8_t position        = OLED_HEIGHT * currentScreen / scrollContentSize;
       if (lastValue)
         scrollBlink = !scrollBlink;
-      if ((!lastValue || !scrollBlink) && !scrollingDown)
+      if (!lastValue || !scrollBlink)
         OLED::drawScrollIndicator(position, indicatorHeight);
     } else {
       // Draw description
@@ -1120,15 +1157,8 @@ void gui_Menu(const menuitem *menu) {
     }
 
     if (lcdRefresh) {
-      if (scrollingDown) {
-        OLED::maskScrollIndicatorOnOLED();
-        OLED::transitionScrollDown();
-        scrollingDown = false;
-        animOpenState = false;
-      } else {
       OLED::refresh(); // update the LCD
       osDelay(40);
-      }
       lcdRefresh = false;
     }
 
@@ -1139,6 +1169,16 @@ void gui_Menu(const menuitem *menu) {
       lastButtonState        = buttons;
     }
 
+    auto callIncrementHandler = [&]() {
+      wasInGuiMenu = false;
+      bool res     = menu[currentScreen].incrementHandler();
+      if (wasInGuiMenu) {
+        navState = NavState::Exiting;
+      }
+      wasInGuiMenu = true;
+      return res;
+    };
+
     switch (buttons) {
     case BUTTON_BOTH:
       earlyExit = true; // will make us exit next loop
@@ -1148,7 +1188,7 @@ void gui_Menu(const menuitem *menu) {
       // increment
       if (scrollMessage.isReset()) {
         if (menu[currentScreen].incrementHandler != nullptr) {
-          lastValue = menu[currentScreen].incrementHandler();
+          lastValue = callIncrementHandler();
         } else {
           earlyExit = true;
         }
@@ -1158,14 +1198,14 @@ void gui_Menu(const menuitem *menu) {
     case BUTTON_B_SHORT:
       if (scrollMessage.isReset()) {
         currentScreen++;
-        scrollingDown = true;
+        navState  = NavState::ScrollingDown;
         lastValue = false;
       } else
         scrollMessage.reset();
       break;
     case BUTTON_F_LONG:
       if (xTaskGetTickCount() + autoRepeatAcceleration > autoRepeatTimer + PRESS_ACCEL_INTERVAL_MAX) {
-        if ((lastValue = menu[currentScreen].incrementHandler()))
+        if ((lastValue = callIncrementHandler()))
           autoRepeatTimer = 1000;
         else
           autoRepeatTimer = 0;
@@ -1180,7 +1220,7 @@ void gui_Menu(const menuitem *menu) {
     case BUTTON_B_LONG:
       if (xTaskGetTickCount() - autoRepeatTimer + autoRepeatAcceleration > PRESS_ACCEL_INTERVAL_MAX) {
         currentScreen++;
-        scrollingDown   = true;
+        navState        = NavState::ScrollingDown;
         autoRepeatTimer = xTaskGetTickCount();
         scrollMessage.reset();
 
@@ -1203,8 +1243,6 @@ void gui_Menu(const menuitem *menu) {
       scrollMessage.reset();
     }
   }
-
-  animOpenState = false;
 }
 
 void enterSettingsMenu() {

source/Core/Threads/GUIThread.cpp

@@ -787,6 +787,7 @@ void startGUITask(void const *argument __unused) {
   bool    buttonLockout          = false;
   bool    tempOnDisplay          = false;
   bool    tipDisconnectedDisplay = false;
+  bool    showExitMenuTransition = false;
   {
     // Generate the flipped screen into ram for later use
     // flipped is generated by flipping each row
@@ -855,6 +856,10 @@ void startGUITask(void const *argument __unused) {
       break;
     case BUTTON_B_SHORT:
       enterSettingsMenu(); // enter the settings menu
+      {
+        OLED::useSecondaryFramebuffer(true);
+        showExitMenuTransition = true;
+      }
       buttonLockout = true;
       break;
     default:
@@ -960,7 +965,13 @@ void startGUITask(void const *argument __unused) {
       }
     }
 
+    if (showExitMenuTransition) {
+      OLED::useSecondaryFramebuffer(false);
+      OLED::transitionSecondaryFramebuffer(false);
+      showExitMenuTransition = false;
+    } else {
       OLED::refresh();
       GUIDelay();
     }
+  }
 }