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 23 Jun 2008 02:56:41 PM UTC  
 
Category: NonePriority: 3 - Low
Status: DonePrivacy: Public
Assigned to: Claudio Fontana <sick_soul>Open/Closed: Open

Wed 25 Jun 2008 09:58:01 PM UTC, comment #4:

applied, available in latest CVS build.

Claudio Fontana <sick_soul>
Project AdministratorIn charge of this item.
Tue 24 Jun 2008 10:25:09 PM UTC, 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 23 Jun 2008 05:02:07 PM UTC, 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 23 Jun 2008 04:14:18 PM UTC, 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 23 Jun 2008 02:56:41 PM UTC, 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 25 Jun 2008 09:58:01 PM UTCsick_soulStatusIn Progress=>Done
    Tue 24 Jun 2008 10:25:09 PM UTCsick_soulPriority5 - Normal=>3 - Low
      StatusNone=>In Progress
      Assigned toNone=>sick_soul
    Mon 23 Jun 2008 02:56:41 PM UTCntdAttached File-=>Added default-choice.patch, #15916
      Carbon-Copy-=>Added ntd

    Back to the top


    Powered by Savane 3.1-cleanup