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