<Swing Dev> RFR 8232880: Update test documentation with additional settings for client UI tooltip tests
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Fri Nov 1 20:58:55 UTC 2019
+1
On 11/1/19 7:41 am, Alexey Ivanov wrote:
> Thank you, Dmitry!
> The changes look good to me.
>
> On 01/11/2019 13:09, Dmitry Markov wrote:
>> Hi Alexey,
>>
>> I have updated the fix. Please find the new version here: http://cr.openjdk.java.net/~dmarkov/8232880/webrev.03/
>>
>> Thanks,
>> Dmitry
>>
>>> On 31 Oct 2019, at 16:27, Alexey Ivanov <alexey.ivanov at oracle.com <mailto:alexey.ivanov at oracle.com>> wrote:
>>>
>>> Hi Dmitry,
>>>
>>> 437 …by the operating system. …
>>>
>>> I'd modify the following text a bit:
>>> To run the test correctly, the default global key shortcut should be disabled. Follow the steps above, and then deselect "Turn keyboard access on or off" property which is responsible for `CTRL + F1` combination.
>>>
>>> Does it sound clearer?
>>> I'd not use backticks on the "Turn keyboard access on or off" because it's not something user is typing, nor is it a piece of code. Is the word “property” correct? Does “shortcut” or “option” fit better?
>>>
>>> I'd recommend adding quotes around the option to look for:
>>> 448 in the right-side pane look for "Turn off Windows key hotkeys" and double click on it;
>>>
>>> Consider adding an empty line before this line
>>> 450 Note: restart is required to make the settings take effect.
>>> to make it a separate paragraph in HTML.
>>>
>>>
>>> Regards,
>>> Alexey
--
Best regards, Sergey.
More information about the swing-dev
mailing list