From c982ae7079e88bb2519222bb432bf71ba6cc635f Mon Sep 17 00:00:00 2001 From: Glenn Maynard Date: Wed, 11 Apr 2007 22:20:35 +0000 Subject: [PATCH] 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. --- stepmania/src/OptionsList.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/stepmania/src/OptionsList.cpp b/stepmania/src/OptionsList.cpp index bc431b3bb7..81fdaf5ed0 100644 --- a/stepmania/src/OptionsList.cpp +++ b/stepmania/src/OptionsList.cpp @@ -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 );