Skip to content

Commit

Permalink
Fix legacy config migration not defaulting to showing Action Bar for …
Browse files Browse the repository at this point in the history
…the first dashboard overlay
  • Loading branch information
elvissteinjr committed Oct 22, 2024
1 parent 360a73a commit 71755e2
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions src/Shared/ConfigManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -992,6 +992,12 @@ void ConfigManager::MigrateLegacyOverlayProfileFromConfig(Ini& config, bool appl
config.WriteString(section.c_str(), "Tags", tag_str.c_str());
}

//0-Dashboard overlay had no Floating UI, always only showing the Action Bar, but didn't make use of the property value which defaulted and stayed false, so force it to true
if (overlay_id == 0)
{
config.WriteBool(section.c_str(), "ShowActionBar", true);
}

//Write Display Mode to new config string (0 is always Desktop+ tab)
config.WriteInt(section.c_str(), "DisplayMode", (overlay_id == 0) ? ovrl_dispmode_dplustab : config.ReadInt(section.c_str(), "DetachedDisplayMode", ovrl_dispmode_always) );

Expand Down

0 comments on commit 71755e2

Please sign in to comment.