When an option on a SELECT_ONE row is chosen, don't change focus
to "exit". Only a couple rows are SELECT_ONE; the rest are SELECT_MULTIPLE. Selecting "exit" automatically changes the flow of the menu on these rows to be unlike the majority, which makes them clumsy to use: you press start, then start tapping right to go to "exit" before realizing it's already put you there (and now you've moved off it). This makes the menus consistent, which makes the menu much easier to manipulate intuitively.
This commit is contained in:
@@ -675,6 +675,7 @@ bool OptionsList::Start()
|
||||
|
||||
SelectItem( GetCurrentRow(), m_iMenuStackSelection );
|
||||
|
||||
#if 0
|
||||
if( pHandler->m_Def.m_selectType == SELECT_ONE )
|
||||
{
|
||||
/* Move to the exit row. Don't exit, so different entries don't have widely
|
||||
@@ -683,6 +684,7 @@ bool OptionsList::Start()
|
||||
m_iMenuStackSelection = (int)bSelections.size();
|
||||
PositionCursor();
|
||||
}
|
||||
#endif
|
||||
|
||||
Message msg("OptionsListStart");
|
||||
msg.SetParam( "Player", m_pn );
|
||||
|
||||
Reference in New Issue
Block a user