This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH setup 00/13] ListView Package Chooser

On 8/5/2018 6:08 PM, Jon Turney wrote:
Drag setup into the 1990s, by replacing the custom-drawn package chooser
with a ListView common control.

As well as removing a lot of Win32 GDI drawing, this also enables the
following improvements to be much more straightforward:

* Use standard UI elements to choose an action to take on a package or
category, rather than the weird UX of clicking to cycle around a list of
options of undisclosed length.

This is a huge improvement over the existing UX.

* Add tooltips (initially, the ldesc as a tooltip for sdesc)

This is also a big improvement.  It's nice to see ldesc finally being used.

I have one minor fix (patch attached) and one suggestion: It would be good, if possible, for setup to remember the column widths if the user changes them, just as it now remembers the size and position of the chooser window.

Thanks for doing this.  It must have been a lot of work.


Attachment: 0001-Ensure-that-an-installed-packageversion-has-an-ldesc.patch
Description: Text document

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]