funkey/RetroFE
1
0
Fork 0
mirror of https://github.com/FunKey-Project/RetroFE.git synced 2026-01-29 11:05:40 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed VerticalMenu selectedIndex bug (selecting wrong item)

Browse Source
This commit is contained in:
emb 2015-02-26 12:35:58 -06:00
parent 7f1f6137b6
commit 97635859d5
1 changed files with 16 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
RetroFE/Source/Graphics/PageBuilder.cpp
View File

@ -629,6 +629,9 @@ void PageBuilder::BuildVerticalMenu(ScrollingList *menu, xml_node<> *menuXml, xm
points->push_back(viewInfo);
tweenPoints->push_back(CreateTweenInstance(component));
height += viewInfo->GetHeight();
// increment the selected index to account for the new "invisible" menu item
selectedIndex++;
}
while(!end)
{
@ -679,16 +682,22 @@ void PageBuilder::BuildVerticalMenu(ScrollingList *menu, xml_node<> *menuXml, xm
tweenPoints->push_back(CreateTweenInstance(component));
}
if(selectedIndex >= ((int)points->size()-2))
if(selectedIndex >= ((int)points->size()))
{
//todo: print debug statements when out of range
selectedIndex = 1;
}
else
{
menu->SetSelectedIndex(selectedIndex+1);
std::stringstream ss;
ss << "Design error! Selected menu item was set to " << selectedIndex
<< " although there are only " << points->size()
<< " menu points that can be displayed";
Logger::Write(Logger::ZONE_ERROR, "Layout", "Design error! \"duration\" attribute");
selectedIndex = 0;
}
menu->SetSelectedIndex(selectedIndex);
menu->SetPoints(points, tweenPoints);
}