|
Rockbox mail archiveSubject: Re: CONTEXT_SETTINGS reworking..Re: CONTEXT_SETTINGS reworking..
From: Jonathan Gordon <jdgordy_at_gmail.com>
Date: Sat, 19 Aug 2006 21:48:55 +1000 right, this is hopefully final (iriver only because sdl is broken and i have no sim) On 19/08/06, Jonathan Gordon <jdgordy_at_gmail.com> wrote: > grr.. no, ACTION_SETTINGS_EXIT and _SELECT and _RESET are bad.... use > ACTION_STD_OK and CANCEL and if needed make a new action for screens > that need reset... > > other than that the patch does the patch in the last meg look workable? > > On 19/08/06, Jonathan Gordon <jdgordy_at_gmail.com> wrote: > > replying to myself here :'( > > > > Does this new version look fine? (only done the iriver keymaps, but > > ill do the rest if this is OK'ed) > > > > On 17/08/06, Jonathan Gordon <jdgordy_at_gmail.com> wrote: > > > hey, > > > After the select widget was removed the CONTEXT_SETTINGS context > > > became almost redundant. I had a look in the sources and the only > > > places that it is now used is in alarm, bookmarks, debug_menu, eq and > > > colour chooser screens. > > > > > > What I have done (and want to get checked before its commited) is give > > > any screen that would use CONTEXT_SETTINGS its own context which will > > > usually just remap _SETTINGS_ACCEPT, _RESET and _EXIT to work on that > > > screen, then it will chain to either the regular settings keymap, or > > > chain to the keymap where left is increase instead of next. > > > > > > I have attached the patch to the email (I hope it works) which does > > > this and has the mappings for iriver as an example. > > > > > > Last thing, the debug screen still uses context_settings, and im happy > > > leaving that, but we can add a context for debug, but i dont tinhk its > > > really that important. > > > > > > Thoughts? > > > > > > > > > > > > > > > >
Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |