<Swing Dev> [9] Review request for 8158566 Provide a Swing property to not close toggle menu items on mouse click
Semyon Sadetsky
semyon.sadetsky at oracle.com
Mon Jun 27 11:42:03 UTC 2016
On 6/27/2016 12:47 PM, Alexander Scherbatiy wrote:
> On 27/06/16 10:56, Semyon Sadetsky wrote:
>> On 6/24/2016 3:33 PM, Alexandr Scherbatiy wrote:
>>
>>>
>>> Could you review the updated fix:
>>> http://cr.openjdk.java.net/~alexsch/8158566/webrev.02
>>>
>>> On 6/20/2016 10:55 PM, Phil Race wrote:
>>>> 59 * Some times it is required to select several check box menu
>>>> items from a button
>>>> 60 * group. In this case it is useful that clicking on one check
>>>> box menu item
>>>> 61 * does not close the menu. Such behavior can be controlled by
>>>> the Look and Feel
>>>> 62 * property named {@code "CheckBoxMenuItem.closeOnMouseClick"}.
>>>>
>>>> Just to make things clear, insert here : "The default value is
>>>> {@true}."
>>> Updated.
>>>>
>>>> Setting
>>>> 63 * the property to {@code false} prevents the menu from
>>>> closing when it is clicked
>>>> 64 * by the mouse.
>>>> 65 * Note: some {@code L&F}s may ignore this property.
>>>>
>>>> It appears to me that all L&Fs that inherit from Basic should
>>>> inherit this behaviour.
>>>> Can we safely say something like :
>>>> "All built-in L&Fs and all L&Fs that subclass the BasicLookAndFeel
>>>> will inherit this behaviour".
>>>>
>>>> Otherwise its untestable.
>>> L&Fs which subclass the BasicLookAndFeel and set its own mouse
>>> handler will not automatically handle the closeOnMouseClick property.
>>> That is why the Motif UI has been updated.
>>>
>>> I left the suggested phrase as: "* Note: some {@code L&F}s may
>>> ignore this property. All built-in L&Fs inherit this behaviour."
>> "L&Fs" produces compilation error because the "&" is a prefix for
>> special symbols codes.
>> You need to replace it with "&"
> Updated: http://cr.openjdk.java.net/~alexsch/8158566/webrev.03/
Looks good to me.
--Semyon
>
> Thanks,
> Alexandr.
>>
>> --Semyon
>>>
>>> Thanks,
>>> Alexandr.
>>>>
>>>>
>>>> -phil.
>>>>
>>>> On 06/20/2016 10:59 AM, Alexandr Scherbatiy wrote:
>>>>>
>>>>> Hello,
>>>>>
>>>>> Could you review the updated fix:
>>>>> http://cr.openjdk.java.net/~alexsch/8158566/webrev.01/
>>>>>
>>>>> The documentation is added to the JCheckBoxMenuItem and
>>>>> JRadioButtonMenuItem classes.
>>>>>
>>>>> Thanks,
>>>>> Alexandr.
>>>>>
>>>>> On 6/19/2016 9:05 PM, Philip Race wrote:
>>>>>> So how does one know of the existence of this property ?
>>>>>> There seems to be a documentation/specification element missing
>>>>>> here.
>>>>>>
>>>>>> Perhaps if you query the supported properties on the component
>>>>>> it is returned, but you still need documentation to explain how
>>>>>> it is used/interpreted.
>>>>>>
>>>>>> If it is not documented .. it can not be supported API, which is
>>>>>> what we
>>>>>> need here.
>>>>>>
>>>>>> If the answer is that none of the others are documented,
>>>>>> then that is a problem too .. no matter how long standing.
>>>>>>
>>>>>> -phil.
>>>>>>
>>>>>>
>>>>>> On 6/14/16, 12:49 PM, Alexandr Scherbatiy wrote:
>>>>>>>
>>>>>>> Hello,
>>>>>>>
>>>>>>> Could you review the fix:
>>>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8158566
>>>>>>> webrev: http://cr.openjdk.java.net/~alexsch/8158566/webrev.00
>>>>>>>
>>>>>>> The proposed "RadioButtonMenuItem.closeOnMouseClick" and
>>>>>>> "CheckBoxMenuItem.closeOnMouseClick" properties allows to
>>>>>>> control JRadioButtonMenuItem and JCheckBoxMenuItem closing on
>>>>>>> mouse click.
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Alexandr.
>>>>>>>
>>>>>>>
>>>>>
>>>>
>>>
>>
>
More information about the swing-dev
mailing list