+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.iva...@oracle.com 
<mailto:alexey.iva...@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.

Reply via email to