Re: [cp-patches] FYI: JFileChooser fix

[Lillian Angel]

On Wed, 2006-01-04 at 23:26 +0000, Roman Kennke wrote:
> 
>    }
> @@ -681,7 +683,8 @@
>  
>      retval = ERROR_OPTION;
>  
> -    d.pack();
> +    Insets i = d.getInsets();
> +    d.setSize(500 + i.top + i.bottom, d.getPreferredSize().height);

For width, shouldn't we use the left/right insets instead of top/bottom?

>      d.show();
>      return retval;
> 

Also, the height is slightly off. All 7 'visible rows' should be
visible, only 6 are fully visible and the 7th is cut off.


Thanks,
Lillian