Hi Alexey,

I have updated the fix. Please find the new version here: 
http://cr.openjdk.java.net/~dmarkov/8232880/webrev.03/ 
<http://cr.openjdk.java.net/~dmarkov/8232880/webrev.03/>

Thanks,
Dmitry

> On 31 Oct 2019, at 16:27, Alexey Ivanov <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
> 
> On 31/10/2019 10:56, Dmitry Markov wrote:
>> Hi Alexey,
>> 
>> I have updated the fix based on your recommendation. The new version is 
>> located at: http://cr.openjdk.java.net/~dmarkov/8232880/webrev.02/
>> Also please find my answers inline.
>> 
>> Thanks,
>> Dmitry
>> 
>>> On 29 Oct 2019, at 19:29, Alexey Ivanov <alexey.iva...@oracle.com 
>>> <mailto:alexey.iva...@oracle.com>> wrote:
>>> 
>>> Hi Dmitry,
>>> 
>>> Shall we drop hyphen in the header: “Client UI Tests”?
>>> 
>>> I think there should be no definite article in this sentence: “use -the- 
>>> key sequences”.
>>> 
>>> “…Turn off Windowss key hotkeys…”, there's an extra ‘s’ in Windows.
>>> 
>>> ”Note: restart is required to make the settings take effect.”
>>> Just to confirm: is signing out and signing in not enough?
>> According to Microsoft site: restart is required but I guess signing out/in 
>> should work too. Unfortunately I do not have Windows on hand to check it out.
>> 
>>> 
>>> I'd use backticks for gpedit markup: “Type `gpedit` in the Search…”
>>> 
>>> 
>>> Does it make sense to move the example into macOS section? Then the steps 
>>> to disable the shortcut can be reduced to the required option only. The 
>>> steps themselves should not be listed as code, i.e. no backticks.
>>> 
>>> (For my understanding, "Turn keyboard access on or off" turns off only one 
>>> specific shortcut, i.e. Ctrl+F1?)
>> Yes, that’s right. I have added clarification to the doc.
>> 
>>> 
>>> 
>>> Regards,
>>> Alexey

Reply via email to