patchGNU Source Installer - Patches: patch #6550, "Choose" as default...

 
 

You are not allowed to post comments on this tracker with your current authentification level.

patch #6550: "Choose" as default button while selecting packages

Submitted by:  Nicola Fontana <ntd>
Submitted on:  Mon Jun 23 14:56:41 2008  
 
Category: NonePriority: 3 - Low
Status: DonePrivacy: Public
Assigned to: Claudio Fontana <sick_soul>Open/Closed: Open

Wed Jun 25 21:58:01 2008, comment #4:

applied, available in latest CVS build.

Claudio Fontana <sick_soul>
Project AdministratorIn charge of this item.
Tue Jun 24 22:25:09 2008, comment #3:

I thought about this for a while, and it does feel better
this way. I am integrating this change.

Claudio Fontana <sick_soul>
Project AdministratorIn charge of this item.
Mon Jun 23 17:02:07 2008, comment #2:

Ok, next time I'll use a proper patch format.

I like to have a default button because of user interface consistency. With the default keybindings you could still enter in the directories using <SHIFT><SPACE>, although I don't know why it is not cited in the documentation.

Nicola Fontana <ntd>
Mon Jun 23 16:14:18 2008, comment #1:

As for the diff format, the best way to submit patches
to me for inclusion would be to make p0 or p1 unified
diffs against the latest CVS tree, with top srcdir
sourceinstall2, using diff -Naupr

But for out of tree patches restricted to a certain
component (example: sourceinstall-gtk), it is probably
ok to generate the changeset by setting the
top srcdir to the specific component directory, as
users can apply it easily to the official releases.

I think that in any case it would be better not to
include the .po changes, as those changes can make it
difficult to apply the patch to a later version with
different line numbers in the .po files.
Those files should be automatically updated during
the build process anyway.

Regarding the feature itself, I find it useful to
start browsing directories with the arrow keys
and ENTER, and this change would choose the
directory instead. But I can see your use case too.

Claudio Fontana <sick_soul>
Project AdministratorIn charge of this item.
Mon Jun 23 14:56:41 2008, original submission:

It'll be useful to have [Choose] as the default response: this allows to double-click on the package name or press <ENTER>.

I attached a patch (hopefully in the right format) that does the job.

Nicola Fontana <ntd>

 

Attached Files
file #15916:  default-choice.patch added by ntd (4kB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by sick_soul (Posted a comment)
  • -unavailable- added by ntd (Submitted the item)
  • -unavailable- added by ntd
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed Jun 25 21:58:01 2008sick_soulStatusIn Progress=>Done
    Tue Jun 24 22:25:09 2008sick_soulPriority5 - Normal=>3 - Low
      StatusNone=>In Progress
      Assigned toNone=>sick_soul
    Mon Jun 23 14:56:41 2008ntdAttached File-=>Added default-choice.patch, #15916
      Carbon-Copy-=>Added ntd

    Back to the top


    Powered by Savane 3.1-cleanup