Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Manual
  • 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 Johannes Linke - 2009-08-05
Last edited by Dominik Riebeling - 2011-06-05

FS#10491 - Add keyclicks to manual

his is only the first shot of adding the setting keyclicks to the manual. my english is not the best, so the description might need improvements.

things i would like to ask: is it necessary to metnion “this menu relates to keyclicks”? the manual says this for other submenus, but in nearly all cases it’s obsolete, of course keyclicks relates to keyclicks…

and is it necessary to say some general things about this setting (this setting controls…) and then explain every option (if set to off….)? leaving the first part out would shorten some descriptions without making them harder to understand.

there some code style corrections in it, i tried to unify this file a bit. the real changes are at the end of the file. you dont need to and i know you wont include the corrections in the commit :)

Closed by  Dominik Riebeling
2011-06-05 10:28
Reason for closing:  Accepted
Additional comments about closing:  

committed with minor modifications.

Alex Parker commented on 2009-08-10 09:03

If you know that we don’t like unrelated things in one patch, then why include them? It only makes me not want to look through this as it takes ages to work out what are real changes and what aren’t. Please attach a new patch with only the actual changes in it.

Johannes Linke commented on 2009-08-10 09:27

if i attach a separate patch with style corrections, you say i should do that when i’m working on this file. if i’m including style corrections in a patch when i’m working on a file you say i should make a separate patch. wtf?

i’m really sorry that i don’t know when wich changes in which patch are appreciated.

Alex Parker commented on 2009-08-11 08:46

I know in the past I haven’t committed style only changes as it didn’t seem worthwhile (for those changes), but I appear to have confused you slightly. I have no objection to two patches here, one style and one “real” - it is just when a patch is full of both, I find it difficult to tell which are which :) Sorry for the confusion, and thanks for working on this.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing