Hi Petr,
please see in-line:
On 01/23/2014 07:25 PM, Petr Pchelko wrote:
Hello, Alexander.
Does your fix also address https://bugs.openjdk.java.net/browse/JDK-7124365 ?
As I understand, it’s not?
Yes, it isn't. From my understanding, there is no way to maximize window
to size smaller than
screen size (excluding dock and menubar) on Mac OS. Such windows are
considered as not maximized.
Javadoc says:
* Note, the given maximized bounds are used as a hint for the native
* system, because the underlying platform may not support setting the
* location and/or size of the maximized windows.
I think that Mac OS is the case when this feature is not supported and
7124365 may be closed as not an issue.
Could you please update the copyright years.
Done in place.
I have a concern about Window.setMinimumSize update. Why are you adding it?
This is done for following case:
We have a frame with
f.setMinimumSize(new Dimension(200, 200));
f.setMaximumSize(new Dimension(100, 100));
Call to f.setMinimumSize(null); will remove minimum size hints and only
maximum size hints (100x100) should be applied,
but window will remain 200x200 size until next move or resize. So here
we resize window to fit the maximum size immediately.
XDecoratedPeer.java has some space-only changes.
It is intentional.
Also it would be great to have a couple of tests)
I'll add some tests later.
Thanks,
Alexander.
With best regards. Petr.
23 янв. 2014 г., в 6:55 после полудня, Alexander Zvegintsev
<[email protected]> написал(а):
Hello AWT team,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/6464548/webrev.00/
for
https://bugs.openjdk.java.net/browse/JDK-6464548 Reopen
https://bugs.openjdk.java.net/browse/JDK-6464551 XAWT: Frame.setMaximizedBounds
restricts resizing
setMinimumSize() for java.awt.Window was implemented in JDK 6. I think it is
time to implement setMaximumSize at least in JDK 9 :)
--
Thanks,
Alexander.
