From e13425d2318ad98c3aea67d3345f321dcc2aab0d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89milie=20Feral?= Date: Thu, 19 Dec 2019 15:41:40 +0100 Subject: [PATCH] [apps] Add comment about LED color in Dutch exam mode --- apps/apps_container.cpp | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/apps/apps_container.cpp b/apps/apps_container.cpp index 8d96b2b02..5656aecb9 100644 --- a/apps/apps_container.cpp +++ b/apps/apps_container.cpp @@ -327,6 +327,13 @@ void AppsContainer::redrawWindow() { void AppsContainer::activateExamMode(GlobalPreferences::ExamMode examMode) { assert(examMode == GlobalPreferences::ExamMode::Standard || examMode == GlobalPreferences::ExamMode::Dutch); reset(); + /* The Dutch exam mode LED is supposed to be orange but we can only make + * blink "pure" colors: with RGB leds on or off (as the PWM is used for + * blinking). The closest "pure" color is Yellow. Moreover, Orange LED is + * already used when the battery is charging. Using yellow, we can assert + * that the yellow LED only means that Dutch exam mode is on and avoid + * confusing states when the battery is charging and states when the Dutch + * exam mode is on. */ Ion::LED::setColor(examMode == GlobalPreferences::ExamMode::Dutch ? KDColorYellow : KDColorRed); Ion::LED::setBlinking(1000, 0.1f); }