Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface → Themes
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by punkt - 2010-02-08
Last edited by jdgordon - 2010-02-24

FS#10982 - New WPS tag: %Tk

This patch adds a new tag %Tk. This works exactly the same as %Tl, except that it turns true on touchscreen release instead of touchscreen press. This can be used to implement a WPS where the first touch displays a viewport with buttons, and a second touch then activates a button.

A modified version of the Fidelity theme which I use to test this feature can be found here:
http://www.theess.de/assets/files/Fidelity_Tk.zip

Link to the discussion on rockbox-dev:
http://www.rockbox.org/mail/archive/rockbox-dev-archive-2010-02/0022.shtml

   wps_tk.patch (3.6 KiB)
Closed by  jdgordon
2010-02-24 06:19
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

thanks! in r24876.

jdgordon commented on 2010-02-09 05:36

do you want to try getting %Tl working better? or should we just accept this patch?

punkt commented on 2010-02-09 10:46

I’d like to look into %Tl this weekend. I recommend putting the %Tk patch on hold until then.

punkt commented on 2010-02-13 23:24

Here’s another take on the matter. As stated in the discussion on the mailing list, my intent is to provide a functionality similar to the original D2 firmware, where the first touch displays touch buttons, which can then be operated using subsequent touches. %Tl in its current form is insufficient: when I touch the display, the buttons are displayed, and the button at the position I touched is immediately triggered. To work around this issue, I made the %Tk patch which my theme uses to show the buttons on a touch release.

The new patch takes a different approach: It modifies the behaviour of touch buttons to be more similar to the way gui buttons operate in desktop applications. Upon a touch press event, the button at the touch position is armed. Upon a touch repeat or release, the button at the touch position is triggered only if it is armed. Upon release (and wps entry), all buttons are disarmed. E.g. when you touch press on an empty area, then while pressing drag your finger on a button, then release the button, the button is not triggered.

The new patch obsoletes the %Tk patch, as the behaviour I want to have can now be achieved using %Tl.

   touchregion_armed.patch (4.1 KiB)
punkt commented on 2010-02-14 02:00

Here’s the theme used for testing the new patch: http://www.theess.de/assets/files/Fidelity_Tl.zip

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing