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 3/3] Add the last element of URL path to site chooser, if interesting.

On Tue, Nov 09, 2010 at 02:21:23PM +0000, Jon TURNEY wrote:
>On 08/11/2010 05:54, Christopher Faylor wrote:
>> On Sun, Nov 07, 2010 at 03:54:50PM +0000, Jon TURNEY wrote:
>>> On 06/11/2010 18:00, Christopher Faylor wrote:
>>>> This is great, please check in.  I suspect that you're just trying a
>>>> local mirror or something but if there are actually sites in the mirror
>>>> list which are not proper FQDNs, please report them so that they can be
>>>> fixed in the mirror database.
>>> Yes, you are correct.  There are no sites in the mirror list which aren't
>>> FQDNs. I was just fixing what happens when you add 'http://localhost/cygwin/'
>>> (or some other locally-known non-FQDN) to the mirror list.
>>> Just to be clear, you want me to check in the whole patch, and not just that fix?
>> Actually, on reflection, I don't see a great benefit to parsing the UFL.
>> Why not just list the whole thing?  What would be more useful to the
>> Cygwin project would be to list the other information like country.
>As a start, the attached patch removes the URL parsing and just shows the
>whole URL in the site chooser.
>I think displaying multiple columns of information would require a grid
>control of some sort.

I wanted to like this but the mirror box is too small to show all of the
long urls so it doesn't really solve the problem of providing the user
with enough information to make a selection.  I guess we could make the
box much bigger but I wonder if that would be adequate.

I guess I'll look at your original patch next to see how that does things.


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