RFR 8232880: Update test documentation with additional settings for client UI tooltip tests

[Dmitry Markov]

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.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
> 
> 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.ivanov at oracle.com <mailto:alexey.ivanov at 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