<html> <head> <base href="https://bugs.documentfoundation.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - LibO doesn’t obey Windows OS setting “only show the accelerator underline when the Alt key is being pressed”" href="https://bugs.documentfoundation.org/show_bug.cgi?id=54169#c56">Comment # 56</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - LibO doesn’t obey Windows OS setting “only show the accelerator underline when the Alt key is being pressed”" href="https://bugs.documentfoundation.org/show_bug.cgi?id=54169">bug 54169</a> from <a class="email" href="mailto:mikekaganski@hotmail.com" title="Mike Kaganski <mikekaganski@hotmail.com>"> Mike Kaganski</a> <pre>(In reply to Samuel Mehrbrodt (CIB) from <a href="show_bug.cgi?id=54169#c18">comment #18</a>) > So, on Windows we need to check the registry for HKEY_CURRENT_USER\Control > Panel\Accessibility\Keyboard Preference > > If the string value "On = 1", mnemonics should always be shown. Else they > should only be shown when the alt key is pressed. Just a mention that we should not use a registry read to get the setting. Rather, a call to SystemParametersInfoW [1] is needed to get the value of SPI_GETKEYBOARDCUES, which is the setting we are looking for. Commit 53df91adab37a65d87a44aa724b28128f32e9de5[2] had removed the mbEnableAccel, which was the thing wrongly handled in the previous commit (see <a href="show_bug.cgi?id=54169#c33">comment 33</a>, reverted in <a href="show_bug.cgi?id=54169#c36">comment 36</a>). So now the change is even easier, only needs to care about mbAutoAccel. But the task is complicated by strange processing inside MenuBarWindow::HandleKeyEvent. For some reason, it makes the underline depend on pressing Esc key, thus after first show of underline caused by Alt, the underline will correctly hide, but then not shown until Esc is pressed. Further, the underline should be also shown in menu when F10 is pressed - generally, on any activation of menu on Windows. But it would only blink currently in that case (also only once, and not blink again until Esc is pressed). [1] <a href="https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-systemparametersinfow">https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-systemparametersinfow</a> [2] <a href="https://git.libreoffice.org/core/+/53df91adab37a65d87a44aa724b28128f32e9de5">https://git.libreoffice.org/core/+/53df91adab37a65d87a44aa724b28128f32e9de5</a></pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>