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

[Alexey Ivanov]

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