Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Version 3.0
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Lamed - 2006-04-19
Last edited by lostlogic - 2006-04-19

FS#5182 - One Button hold for vertical screen scroll patch

This patch is to change the vertical screen scroll keybinds from two buttons macro (i.e ON+LEFT/RIGHT on HX00),
to one long (hold) press (LEFT/RIGHT on Hx00), as discussed a long time ago in IRC.
This will help users to operate the vertical scroll as it demands two hand operation as it is now.

the player isn't effected by this patch as it only has two lines lcd.

The ONDIO is the only player that still uses MENU + LEFT / RIGHT combination, because it's RIGHT_REPEAT button is already occupied with the TREE_CONTEXT menu.
It is better if someone checks the ondio before submitting to CVS because I can't get the SDL sims to work :/

There's one small bug solved in all the lcd-bitmapped drivers that caused the inverse bar to crop down in the middle when the user scrolled vertically.

(note, there is a redundant #define TREE_POWER_BTN in tree.h, my assumption is that it was before the gui_list widget was introduced.)

Closed by  Lamed
2006-08-12 13:43
Reason for closing:  Accepted
Lamed commented on 2006-04-19 19:13

ircnick:sharpe checked an ondio sim for me, the patch is fine :)

Lamed commented on 2006-04-19 19:17
Warning: Undefined array key "useheading" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 1099 Warning: Undefined array key "target" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557 Warning: Trying to access array offset on value of type null in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557

and ho, yeah, someone should put it into the manual as well :)

http://www.rockbox.org/tracker/task/5073

Lamed commented on 2006-04-22 10:37

the line:
xrect = MAX(w - offset, 0);
in all LCD driver should be: xrect = xpos + MAX(w - offset, 0);
please do not commit before changing

Lamed commented on 2006-05-04 04:15
Warning: Undefined array key "useheading" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 1099 Warning: Undefined array key "target" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557 Warning: Trying to access array offset on value of type null in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557

The right, working file. Btw, this also _really_ fixes http://www.rockbox.org/tracker/task/5267 (I was the one who wrote horizontal screen scroll, a long time ago we had an irc conversation that it is better if screen scrolling would not need to use two keys combination, and a short while a go I had time to write it down. It also fixes a behaviour whereas holding RIGHT key on a folder will enter subfolders very quickly, an unwanted behaviour for sure ( BUTTON_RIGHT | BUTTON_REPEAT is removed) )

Project Manager

I have committed the cropping fix. I have no opinion about the key mapping, so I'll probably commit that as well, as I'm a big fan of one-handed operation.

Project Manager

Here's an updated version without the bug fix.

@ Linus:

You should have checked the previous commits before committing this. Peter D'Hoye fixed the inverse bar issue 3 days ago.

The only difference between his fix and your commit is that he didn't clamp the value to zero but let it become negative under certain conditions. That doesn't hurt though, as lcd_fillrect() catches this and returns immediately. His version saved a bit of code…

Project Manager
Warning: Undefined array key "useheading" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 1099 Warning: Undefined array key "target" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557 Warning: Trying to access array offset on value of type null in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557

Yes, he made an attempt to fix it. It was not 100% succesful though, see http://www.rockbox.org/tracker/task/5267.

Lamed commented on 2006-05-16 04:10

Jens, I'm sorry, you're wrong. because he let the numbers get negative, the Icons (on the left) got color-inverted, at least on the h1x0 (which shouldn't really be any diffrent from any other target, I'm just not l00king at the code right now).
Think about it this way: the filling rect from the right side of the text did not 'stopped' at the left side boarder and 'filled' the icon.

Linus, I believe you haven't submitted the key mapping code,
a. why is that?
b. I think it broke the sudoko menu, but I haven't had time to see what caused me the problem (the menu was not responsive at all, I had to shut down).

Project Manager

a) Because I wanted to fix the bug without changing an existing feature.

b) What broke the Sudoku menu?

Lamed commented on 2006-05-21 09:22

b. The Sudoku menu caught menu enter downpresses and was activated on button release (MENU_ENTER became BUTTON_NONE, and MENU_ENTER | BUTTON_REL became MENU_ENTER.)
I'm not sure why is was introduced in such a way, other then the comment saying 'next menu doesn't like release'.
This code isn't necessary with the screen scroll hold button patch for targets that actually uses it, but I assume ONDIO's should still use it : "The ONDIO is the only player that still uses MENU + LEFT / RIGHT combination, because it's RIGHT_REPEAT button is already occupied with the TREE_CONTEXT menu."

I would have checked it myself but I can't get Vmware simulators running… they are all corrupted for me. ##$%#%Y$.
so someone should #ifdef it properly or tell me how to solve the Vmware issue :P

a.huh, I guess I have nothing to say about that.

Project Manager

Any news about the Ondio compatibility with this patch?

Lamed commented on 2006-07-13 12:54

The patch was fine for ondio, but I added one more lastbutton check so that screen scrolling would work on ondio menu system. (without it, pressing menu would have left the menu on click, now it's on release.)

I do remind sudoku_fix.patch is importent for this patchfile.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing