Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface → Simulator
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.4
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by freddyb - 2010-02-12
Last edited by fg - 2010-02-16

FS#11008 - Clickable background and mouse wheel events for UI Simulator

This patch turns the simulator background into a clickable area. i.e. you click on the play button to press play. It also maps mouse scrollwheel events to rockbox scrollwheel events. This facilitates testing because you don’t have to move your hand between the mouse and keyboard. There are currently maps for Fuze, iPod, and e200. I would add the others if there is interest.

   uisim-clickable.diff (5.1 KiB)
Closed by  fg
2010-02-16 17:35
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

Committed as r24696

Admin
fg commented on 2010-02-12 20:52

Two comments after quickly looking at the patch
- Touchscreen targets can have real buttons too
- I’d go for a rectangle for each button instead of this circular area with fixed radius, or at least for a configurable radius per button

freddyb commented on 2010-02-12 21:47

Small change to make coordinates relative to top left corner of lcd display for mouse clicks when using –debug-wps.

   uisim-clickable-v2.diff (5.9 KiB)
Admin
fg commented on 2010-02-13 13:43

v2 doesn’t actually compile for non-remote targets now

freddyb commented on 2010-02-13 15:00

Added touchscreen support and a button map for Cowon D2.
Current targets: Sansa Fuze and E200, iPod Video, Cowon D2, iAudio X5
Disabled mouse wheel for non-scroll wheel targets.
Added conditionals for remotes.

Reduced button radius a bit.
Frank, I’m leaning towards using radius per button approach because rectangles won’t fit as well for the main left right up down buttons on many players.

   uisim-clickable-v3.diff (8 KiB)
freddyb commented on 2010-02-13 18:45

Changed button struct to include radius values per button.

Added –mapping flag which outputs a template for a struct line when clicking on the background to facilitate creating new button maps.
-Drag the mouse before releasing to get a radius value.

   uisim-clickable-v4.diff (9.6 KiB)
freddyb commented on 2010-02-13 23:11

More targets.

   uisim-clickable-v6.diff (14.7 KiB)
freddyb commented on 2010-02-14 05:53

More targets. I believe this covers all the stable ports and the unstable ports except for Samsung models.

Archos FM Recorder, Archos OndioFM, Archos OndioSP, Archos Player, Archos Recorder, Archos RecorderV2
Cowon D2
Gigabeat F/X, Gigabeat S
iAudio 7, iAudio M3, iAudio M5, iAudio X5
Ipod 1g2g, Ipod 3g, Ipod 4g, Ipod Color, Ipod Mini, Ipod Mini2g, Ipod Nano, Ipod Nano2g, Ipod Video
Iriver H100, Iriver H120, Iriver H300
Mrobe 100, Mrobe 500
Sansa C200, Sansa C200v2, Sansa Clip, Sansa E200, Sansa E200v2

   uisim-clickable-v7.diff (19.8 KiB)
freddyb commented on 2010-02-14 15:36

More Targets: This now includes everything stable and unstable.

   uisim-clickable-v9.diff (20.7 KiB)
freddyb commented on 2010-02-14 23:41

More targets. Now includes everything that compiles.
Added a few lines to docs/UISIMULATOR.
Added –debugbuttons flag that prints the description of the button pressed
Added a #WARNING if no button map is defined.

   uisim-clickable-v10.diff (24.6 KiB)
freddyb commented on 2010-02-15 01:35

Took the #WARNING back out.

   uisim-clickable-v11.diff (24.6 KiB)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing