When performing a manual spell check (F7), using the keyboard is impossible because of duplicate accelerator keys. Alt+G is a common accelerator for Ignore All in many apps, but the Check Grammar check box uses the same accelerator. Pressing Alt+G toggles between the Ignore All and Check Grammar check box, which creates the illusion of toggling between the Close button and Ignore All. The Close and Correct buttons use the same Alt+C accelerators. As Alt+G is common for Ignore All, the accelerator for Check Grammar should be changed.
Confirmed with LibO 4.1 and above. I'm assuming this happened when the dialog was redone.
Created attachment 100530 [details] how the dialog appears when you switch to grammar check (there is a blue highlight around the close button)
Created attachment 100531 [details] spellcheck dialog in 4.0
On pc Debian x86-64 with master sources updated yesterday, I could reproduce this. dialog is defined here: cui/uiconfig/ui/spellingdialog.ui
Suggestion: - Ignore All -> "A" - Add to Dictionary -> "D" - Correct -> "O" (since "C" should be indeed for "Close") Caolan: must we ping UX guys about this or does it only depend on devs?
Created attachment 100577 [details] spellcheck in word 2010 Julien: i think we need to keep consistency with users have gotten used to already, but simply not have the blue highlight appear on the buttons when the grammar checkbox is gotten focus.
Created attachment 100578 [details] spellcheck in kingsoft writer
Caolán McNamara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=da6ea10cd4a673661da016c6e8195f12876e7f73 Resolves: fdo#79707 duplicate acceserator keys in spell checking dialog It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-4-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=2493facca9c8067e6c8db286958d27bf3a13b71a&h=libreoffice-4-4 Resolves: fdo#79707 duplicate acceserator keys in spell checking dialog It will be available in 4.4.0.0.beta2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.