Change program colors to reflect controller requirements. Platform bot and sandwich maker don't need TickPrecise.

Author: Mysticial

SerialPrograms/Source/CommonFramework/Panels/UI/PanelElements.cpp

@@ -10,6 +10,7 @@
 #include <QPushButton>
 #include "CommonFramework/Globals.h"
 #include "CommonFramework/Options/Environment/ThemeSelectorOption.h"
+#include "Controllers/ControllerCapability.h"
 #include "PanelElements.h"
 
 namespace PokemonAutomation{
@@ -50,6 +51,7 @@ CollapsibleGroupBox* make_panel_header(
 
     return header;
 }
+#if 0
 CollapsibleGroupBox* make_panel_header(
     QWidget& parent,
     const std::string& display_name,
@@ -92,6 +94,37 @@ CollapsibleGroupBox* make_panel_header(
 
     return header;
 }
+#endif
+CollapsibleGroupBox* make_panel_header(
+    QWidget& parent,
+    const std::string& display_name,
+    const std::string& doc_link,
+    const std::string& description,
+    const ControllerRequirements& requirements
+){
+    CollapsibleGroupBox* header = make_panel_header(parent, display_name, doc_link, description);
+    QLayout* layout = header->widget()->layout();
+
+    std::string text;
+    do{
+        if (requirements.contains(ControllerFeature::NintendoSwitch_DateSkip)){
+            text = html_color_text("(This program requires advanced RPCs. It requires Serial PABotBase.)", COLOR_RED);
+            break;
+        }
+        if (requirements.contains(ControllerFeature::TickPrecise)){
+            text = html_color_text("(This program requires a tick-precise controller.)", COLOR_PURPLE);
+            break;
+        }
+
+        text = html_color_text("(This program does not have any special requirements.)", COLOR_BLUE);
+    }while (false);
+
+    QLabel* label = new QLabel(QString::fromStdString(text), header);
+    label->setWordWrap(true);
+    layout->addWidget(label);
+
+    return header;
+}
 
 
 

SerialPrograms/Source/CommonFramework/Panels/UI/PanelElements.h

@@ -16,6 +16,8 @@ class QPushButton;
 
 namespace PokemonAutomation{
 
+class ControllerRequirements;
+
 
 
 CollapsibleGroupBox* make_panel_header(
@@ -24,13 +26,22 @@ CollapsibleGroupBox* make_panel_header(
     const std::string& doc_link,
     const std::string& description
 );
+#if 0
 CollapsibleGroupBox* make_panel_header(
     QWidget& parent,
     const std::string& display_name,
     const std::string& doc_link,
     const std::string& description,
     FeedbackType feedback
 );
+#endif
+CollapsibleGroupBox* make_panel_header(
+    QWidget& parent,
+    const std::string& display_name,
+    const std::string& doc_link,
+    const std::string& description,
+    const ControllerRequirements& requirements
+);
 
 
 

SerialPrograms/Source/Controllers/ControllerCapability.h

@@ -69,6 +69,10 @@ class ControllerRequirements{
 public:
     ControllerRequirements(std::initializer_list<ControllerFeature> args);
 
+    bool contains(ControllerFeature feature) const{
+        return m_features.contains(feature);
+    }
+
     //  Check compatibility. If compatible, returns empty string.
     //  Otherwise returns one of the missing features.
     std::string check_compatibility(const std::set<ControllerFeature>& features) const;

SerialPrograms/Source/NintendoSwitch/Framework/NintendoSwitch_SwitchSystemOption.cpp

@@ -13,8 +13,9 @@
 namespace PokemonAutomation{
 namespace NintendoSwitch{
 
-    // constexpr Color COLOR_GREEN2(0xff00aa00);
+// constexpr Color COLOR_GREEN2(0xff00aa00);
 
+#if 0
 Color pick_color(FeedbackType feedback){
     switch (feedback){
     case FeedbackType::NONE:
@@ -28,7 +29,16 @@ Color pick_color(FeedbackType feedback){
     }
     return Color();
 }
-
+#endif
+Color pick_color(const ControllerRequirements& requirements){
+    if (requirements.contains(ControllerFeature::NintendoSwitch_DateSkip)){
+        return COLOR_RED;
+    }
+    if (requirements.contains(ControllerFeature::TickPrecise)){
+        return COLOR_PURPLE;
+    }
+    return COLOR_BLUE;
+}
 
 
 const std::string SwitchSystemOption::JSON_CONTROLLER  = "Controller";

SerialPrograms/Source/NintendoSwitch/Framework/NintendoSwitch_SwitchSystemOption.h

@@ -24,10 +24,12 @@
 #include "Controllers/ControllerDescriptor.h"
 
 namespace PokemonAutomation{
+    class ControllerRequirements;
 namespace NintendoSwitch{
 
 
-Color pick_color(FeedbackType feedback);
+//Color pick_color(FeedbackType feedback);
+Color pick_color(const ControllerRequirements& requirements);
 
 
 class SwitchSystemOption{

SerialPrograms/Source/NintendoSwitch/Framework/UI/NintendoSwitch_MultiSwitchProgramWidget.cpp

@@ -60,7 +60,7 @@ MultiSwitchProgramWidget2::MultiSwitchProgramWidget2(
         descriptor.display_name(),
         descriptor.doc_link(),
         descriptor.description(),
-        descriptor.feedback()
+        descriptor.requirements()
     );
     layout->addWidget(header);
 

SerialPrograms/Source/NintendoSwitch/Framework/UI/NintendoSwitch_SingleSwitchProgramWidget.cpp

@@ -52,7 +52,7 @@ SingleSwitchProgramWidget2::SingleSwitchProgramWidget2(
         descriptor.display_name(),
         descriptor.doc_link(),
         descriptor.description(),
-        descriptor.feedback()
+        descriptor.requirements()
     );
     layout->addWidget(header);
 

SerialPrograms/Source/NintendoSwitch/NintendoSwitch_MultiSwitchProgram.cpp

@@ -79,7 +79,7 @@ MultiSwitchProgramDescriptor::MultiSwitchProgramDescriptor(
     size_t default_switches
 )
     : ProgramDescriptor(
-        pick_color(feedback),
+        pick_color(requirements),
         std::move(identifier),
         std::move(category), std::move(display_name),
         std::move(doc_link),

SerialPrograms/Source/NintendoSwitch/NintendoSwitch_SingleSwitchProgram.cpp

@@ -27,7 +27,7 @@ SingleSwitchProgramDescriptor::SingleSwitchProgramDescriptor(
     ControllerRequirements requirements
 )
     : ProgramDescriptor(
-        pick_color(feedback),
+        pick_color(requirements),
         std::move(identifier),
         std::move(category), std::move(display_name),
         std::move(doc_link),

SerialPrograms/Source/PokemonSV/Programs/Sandwiches/PokemonSV_SandwichMaker.cpp

@@ -28,7 +28,7 @@ SandwichMaker_Descriptor::SandwichMaker_Descriptor()
         "Make a sandwich of your choice.",
         FeedbackType::REQUIRED,
         AllowCommandsWhenRunning::DISABLE_COMMANDS,
-        {SerialPABotBase::OLD_NINTENDO_SWITCH_DEFAULT_REQUIREMENTS}
+        {ControllerFeature::NintendoSwitch_ProController}
     )
 {}