Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • Assigned To
    Tomer Shalev
  • Operating System Another
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.3
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Arago - 2009-08-20
Last edited by Tomer Shalev - 2010-02-24

FS#10552 - Cowon D2: pgup/pgdn on minus/plus buttons in text viewer

I have assigned
+ # define VIEWER_PAGE_UP BUTTON_MINUS
+ # define VIEWER_PAGE_DOWN BUTTON_PLUS
in viewer.rock to scroll the text in the reader to the player with damaged sensor but good display (and on a normal player, I think the hardware scroll buttons would all be more comfortable).

Closed by  Tomer Shalev
2010-02-24 21:40
Reason for closing:  Accepted
Additional comments about closing:  

Committed in r24896. Thanks!
I excluded the 'buttons in hold-mode' key mapping.
Arago, if you still like it to be committed, please add your mapping in a separate FlySpray task, and make it dependent on FS#10698.

Rob Purchase commented on 2009-09-01 20:16

Thanks, that’s a sensible addition. The only thing stopping me from adding this to SVN is that all code contributions to Rockbox should be attributed to a real named person - so please can you fill in the Real Name field on your Flyspray profile.

Arago commented on 2009-10-04 19:23

Sorry, but I dont want to do this. Use my nick instead. And here is new version of this patch (added autoscroll switching using menu button in the Hold mode).

Requires functionality of the buttons in the Hold mode patch by andrewthecoder.
I can’t post it here, because this code is not mine, but you can take it with any config on http://iaudiophile.net/forums/showthread.php?t=28467 or on http://uploads.arago.shell.tor.hu/rockbox/hold_buttons_patch.patch

Tomer Shalev commented on 2010-02-15 06:00
Sorry, but I dont want to do this. Use my nick instead.

Arago, allow me to quote the docs/CONTRIBUTING file:
<quote>

  Credits
  -------
  We believe in crediting all contributors by name. Before committing a patch to
  SVN, we ask that you give us your full real name (no pseudonyms or nicknames)
  for adding to the credits list.

</quote>

You seem to have contributed some really good patches, and we would like to see them being committed.
However, we can not to that without your real name. I don’t think there is any legal workaround for that.
Would you be kind enough to provide a real name so that your work be committed?

I would like to commit your following patches for start:

10552 Cowon D2: pgup/pgdn on minus/plus buttons in text viewe… 10640 Cowon D2: Rockblox game control using hardware buttons
10642 Cowon D2: Rockblox game control using hardware buttons
10643 Cowon D2: Chopper game control using hardware buttons
10644 Cowon D2: JPEG viewer control using hardware buttons an… 10645 Cowon D2: PNG viewer control using hardware buttons and… 10646 Cowon D2: Keyboard text input using hardware buttons an… 10647 Cowon D2: Time screen control using hardware buttons an… 10648 Cowon D2: Mpegplayer control using hardware buttons and…

Regards,
Tomer Shalev

Arago commented on 2010-02-20 21:46

Okay, I can tell you my name, but I do not want to enter it into account information and assign it to my nick - I would like to preserve the anonymity in the Internet. This will be enough?

Tomer Shalev commented on 2010-02-21 03:44
Okay, I can tell you my name, but I do not want to enter it into account information and assign it to my nick - I would like to preserve the anonymity in the Internet. This will be enough?

This will be enough, but please notice that your full name will be shown in docs/CREDITS. I will not write your name in the commit log message, and will commit docs/CREDITS separately.
You can send me your name privately.

Arago commented on 2010-02-21 09:55

Yes, i understand it.
I sent email to address in Your profile.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing