dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: For committal: morse menu patch

For committal: morse menu patch

From: David Johnston <>
Date: Mon, 15 Jun 2009 15:05:58 -0700

Just to make everything totally clear, I'm suggesting this patch to
add to SVN. Comments? Or, is there anything else I need to do to
make this more likely to happen?


On Mon, Jun 15, 2009 at 9:13 AM, David Johnston<> wrote:
> I just put up a new patch, my first forray into RB development.  It's
> pretty straigtforward, simply adding an entry to the menu system to
> set whether the user wants to use the morse code input screen rather
> than the standard one.  I think it would be a good addition to the
> codebase, since it creates a more intuitive way to switch screens, as
> well as saving the user's preference of which screen to use across
> sessions.
> I would welcome feedback on functionality as well as menu item text
> and placement.  So far I've only tested it on h120 but I imagine there
> would be no trouble compiling it for other platforms.  I made an
> effort to follow all the necessary precautions with conditional
> compiling and so on.
> Below is the official description in the Flyspray entry
> (
> -David
> ---
> This patch adds an entry to the settings menus to set whether the user
> wants to use the morse code text input screen rather than the main
> keyboard screen. An entry in the general_settings struct now holds the
> current morse input bool.
> As a result of this change, whenever the user changes between the
> morse and standard text input screens -- either through the menu or
> the key combo while inputting text -- Rockbox will remember that
> setting across sessions. This makes the menu item a somewhat redundant
> feature, but still valuable since it doesn't require remembering an
> obscure key combination to change text input style.
> I applied this patch to revision 21293, but should be generally applicable.
Received on 2009-06-16

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy