<AWT Dev> [9] Review request for 8163101: dual-screen issue with JMenu, JPopupMenu

[Semyon Sadetsky]

On 11/7/2016 7:55 PM, Sergey Bylokhov wrote:
> On 07.11.16 19:48, Semyon Sadetsky wrote:
>>> This is unrelated, the code should depends on bounds/scales/etc from
>>> device where the popup or window are located.
>> Originally, the code was written in the present state. And it has been
>> working fine all the time. Changing the design is out of the scope of
>> this issue. This issue is about regression that is caused by the change
>> which didn't improve design in the way you want it. So, the regression
>> should be fixed, it is mandatory action because the regression was
>> introduced in 9. The rest improvement are optional and may be postponed.
>> Is it clear to you?
>
> It is clear that you spend a lot of time in discussions instead of 
> fixing the bugs assigned to you.
I waste my time on you, not on the bug. I have fixed the bug as it 
should be fixed, and I explained why.
Anyone can improve the code after the fix is pushed in a separate issue 
and if it will be prioritized it may be pushed in 9.
There is a process which we should follow since ZBB date is closer. If 
you want to change the process you should discuss at first.
>>>>>>>> I proposed to do same to investigate the problem. Does it mean 
>>>>>>>> that
>>>>>>>> you
>>>>>>>> approve the current fix?
>>>>>>>
>>>>>>> No, this is one is a separate bug. and I already described why.
>>>>>> But do you have any other concerns to the current fix?
>>>>>
>>>>> The code in question should use the bounds of the screen where the
>>>>> component is located, not a bounds of the main screen.
>>>>>
>>>>>>>
>>>>>>>> But severity of this bug is rather low because it is internal
>>>>>>>> problem
>>>>>>>> that does not affect user experience. It may be deferred to the 
>>>>>>>> next
>>>>>>>> major JDK release.
>>>>>>>>
>>>>>>>> Also note, that the same bug was already closed as "won't fix"
>>>>>>>> before:
>>>>>>>> https://bugs.openjdk.java.net/browse/JDK-5100801
>>>>>>>>
>>>>>>>> --Semyon
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>>
>>
>
>