setup.exe sizes openldap-2-2-15

[Reini Urban]

Attached is my ugly patch to make the dialogs larger by 200, to limit 
the package version string to max 45 chars and to limit the max package 
column width to 500 pixel (better than nothing). My calculation if the 
max column width from the parent window was too fragile. Maybe someone 
wants to try.

If you want to apply this patch you probably need an assignment.
So I would prefer naturally that someone who already did this, picks my 
ideas and writes the straightforward features by his own. (That's why I 
wrote ugly. Don't look at it, when you want to do it by yourself.)

Gary R. Van Sickle schrieb:
>>Gary R. Van Sickle schrieb:
>>
>>>>Unfortunately most of the rc entries are hardcoded and are not 
>>>>calculated at init. I'd prefer to calc them at init and place the 
>>>>ressources then relatively from left, up, right and bottom, 
>>
>>but this 
>>
>>>>looks like a lot of work.
>>>
>>>It is.
>>
>>Ah, I see. Not much work.
> 
> ???

At first sight it looked dramaticly hacked, but then it suddenly made sense.

>>But nevertheless since we have now just another broken 
>>package version string - openldap-2-2-15 - which makes the 
>>"Current" header exceed the window width, I've made some tiny 
>>changes to limit this width to the max window size.
> 
> Makes sense.
> 
>>Shouldn't we also truncate the max package version string 
>>length in setup to something like 30?
> 
> Well, I guess the first step would be ensuring that the version strings
> aren't wrong in the first place.  Beyond that, setup should probably:
> 
> - Place some sort of sensical limit on column widths.
> - Handle strings which exceed the current cell's width by truncating it (as
> it does now), and displaying the full text as a mouseover popup (the yellow
> box with text things) (which it does not do now).

Something like this is up to you or max.
-- 
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: setup_larger.patch
URL: <http://cygwin.com/pipermail/cygwin-apps/attachments/20041026/de3e3cd6/attachment.ksh>