[rfc][icedtea-web] do not align buttons (in dialogs which do) to minimal size
Jiri Vanek
jvanek at redhat.com
Thu Jun 25 08:15:55 UTC 2015
On 06/24/2015 10:30 PM, Omair Majid wrote:
> * Jiri Vanek <jvanek at redhat.com> [2015-06-24 05:53]:
>> Some dialogs are trying to have buttons of same width. I'm not sure it is
>> necessary, but undoubtedly it cause
>> https://bugzilla.redhat.com/show_bug.cgi?id=1231441
>>
>> This patch is fixing it.
>
> The implementation of sizing is broken, but that doesn't mean that we
> should do away with that!
>
> net.sourceforge.jnlp.util.logging.JavaConsole uses an approach based on
> letting the layout manager set the sizes (correctly) instead of
> calculating things explicitly (and getting it wrong). Put the buttons in
> a JPanel with GridLayout (this forces all buttons to be the same size).
I'm aware of this.
> Then put the JPanel in the container where the buttons originally were.
> No more wrong calculations!
Thats why this patch comes in. It removes all wrong calcualtions, and is letting the layout manager
to do it correctly.
>
> This approach is also described here:
> http://stackoverflow.com/questions/11536089/making-all-button-size-same
>
>> Without debugging main methods and one null check, I would like to push this also to 1.6 and 1.5
>
> I don't have the right to NACK this patch since I am not contributing a
> better fix, but please consider keeping the same-button-sizes feature.
I'm working on patch based on the grid layout, which should do this trick for all buttons in all
extensions of securityPanel. However, I had run to some issues, so I could not post it sooner (and
probably not even today). But hopefully soon enough.
Thanx for reply, and thanx for verification I'm going in right direction.
J.
More information about the distro-pkg-dev
mailing list