[icedtea-web] RFC: change download window to a jdialog

Omair Majid omajid at redhat.com
Fri May 2 23:17:05 UTC 2014


* Andrew Azores <aazores at redhat.com> [2014-05-02 16:48]:
> On 05/02/2014 04:34 PM, Omair Majid wrote:
> >The current download progress indicator window in icedtea-web is a
> >JFrame. Some window managers in Linux assume it's an actual application
> >window and try to handle it's size and position in a more appropriate
> >way. A tiling window manager, for example, will stretch it to fill the
> >screen size.
> >
> >The attached patch makes it a JDialog so it's treated as a dialog and
> >it's size and position are respected.

Thanks for the review!

> There are other places that we should probably consider making this
> change as well. For example, the "could not launch" dialog.
> Reproducer - just hit cancel on the security prompt for Elluminate
> or something. At least with i3, that error dialog also appears to
> actually be a frame and is treated as a new application window.

Thanks, I can reproduce this issue.

> Would you care to
> investigate those as well and roll it up into one patch since it's
> all probably the same thing over again? If not, then feel free to
> just push this one alone (with backport too probably) and the rest
> can just be fixed separately.

I have pushed this patch; I will look at the other cases separately. The
issue seems somewhat different.

Thanks,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681


More information about the distro-pkg-dev mailing list