BUG correction in PageBuilder: ReloadableScrollingText arguments were in incorrect order

Signed-off-by: Vincent-FK <[email protected]>

Author: Vincent-FK

RetroFE/Source/Graphics/Component/ReloadableScrollingText.cpp

@@ -28,7 +28,14 @@
 #include <algorithm>
 
 
-ReloadableScrollingText::ReloadableScrollingText(Configuration &config, bool systemMode, bool layoutMode, bool menuMode, std::string type, std::string textFormat, std::string singlePrefix, std::string singlePostfix, std::string pluralPrefix, std::string pluralPostfix, std::string alignment, Page &p, int displayOffset, Font *font, float scaleX, float scaleY, std::string direction, float scrollingSpeed, float startPosition, float startTime, float endTime )
+ReloadableScrollingText::ReloadableScrollingText(Configuration &config, bool systemMode, 
+    bool layoutMode, bool menuMode, 
+    std::string type, std::string textFormat, std::string singlePrefix, 
+    std::string singlePostfix, std::string pluralPrefix, std::string pluralPostfix, 
+    std::string alignment, Page &p, int displayOffset, Font *font, 
+    float scaleX, float scaleY, std::string direction, float scrollingSpeed, 
+    float startPosition, float startTime, float endTime )
+
     : Component(p)
     , config_(config)
     , systemMode_(systemMode)
@@ -504,7 +511,6 @@ void ReloadableScrollingText::loadText( std::string collection, std::string type
 
     while(std::getline(includeStream, line))
     {
-
         // In horizontal scrolling direction, add a space before every line except the first.
         if (direction_ == "horizontal" && !text_.empty( ))
         {

RetroFE/Source/Graphics/PageBuilder.cpp

@@ -778,7 +778,7 @@ void PageBuilder::loadReloadableImages(xml_node<> *layout, std::string tagName,
                 {
                     pluralPostfix = pluralPostfixXml->value();
                 }
-                c = new ReloadableScrollingText(config_, systemMode, layoutMode, menuMode, type->value(), singlePrefix, singlePostfix, pluralPrefix, pluralPostfix, textFormat, alignment, *page, selectedOffset, font, scaleX_, scaleY_, direction, scrollingSpeed, startPosition, startTime, endTime);
+                c = new ReloadableScrollingText(config_, systemMode, layoutMode, menuMode, type->value(), textFormat, singlePrefix, singlePostfix, pluralPrefix, pluralPostfix, alignment, *page, selectedOffset, font, scaleX_, scaleY_, direction, scrollingSpeed, startPosition, startTime, endTime);
                 c->setId( id );
                 xml_attribute<> *menuScrollReload = componentXml->first_attribute("menuScrollReload");
                 if (menuScrollReload &&

RetroFE/Source/Menu/MenuMode.cpp

@@ -26,9 +26,9 @@
 #define SCREEN_HORIZONTAL_SIZE      240 //RES_HW_SCREEN_HORIZONTAL
 #define SCREEN_VERTICAL_SIZE        240 //RES_HW_SCREEN_VERTICAL
 
-#define SCROLL_SPEED_PX             30 //This means no animations but also no tearing effect
-#define FPS_MENU                    50
-#define ARROWS_PADDING				8
+#define SCROLL_SPEED_PX             30
+#define FPS_MENU                    60
+#define ARROWS_PADDING              8
 
 #define MENU_ZONE_WIDTH             SCREEN_HORIZONTAL_SIZE
 #define MENU_ZONE_HEIGHT            SCREEN_VERTICAL_SIZE