<Swing Dev> [9] Review request for 8158566 Provide a Swing property to not close toggle menu items on mouse click
Alexander Scherbatiy
alexandr.scherbatiy at oracle.com
Mon Jun 27 09:47:24 UTC 2016
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/
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