Bug 102044 - Set cursor focus to the end of the mispelt word in Spelling dialog
Summary: Set cursor focus to the end of the mispelt word in Spelling dialog
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
5.3.0.0.alpha0+
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: accessibility, needsDevEval, topicUI
Depends on:
Blocks: Spell-Checking a11y, Accessibility Dialog
  Show dependency treegraph
 
Reported: 2016-09-10 14:03 UTC by Yousuf Philips (jay) (retired)
Modified: 2023-11-02 18:22 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yousuf Philips (jay) (retired) 2016-09-10 14:03:27 UTC
When the spelling dialog is opened and when a new mispelt word is highlighted, the cursor should return to the 'not in dictionary' text area so that a user can easily modify the word and a screen reader can read the mispelt word, as presently a a11y users cant know which of the text area words are mispelt.
Comment 1 Alex ARNAUD 2017-07-18 16:16:18 UTC
Dear all,

I agree with Yousuf but it's more logical to read the word in most language from left to right so I prefer to have the cursor at the beginningo
Comment 2 Alex ARNAUD 2017-07-18 16:17:20 UTC
(In reply to Alex ARNAUD from comment #1)
> Dear all,
> 
> I agree with Yousuf but it's more logical to read the word in most language
> from left to right so I prefer to have the cursor at the beginningo

...beginning of the misspell word.

Best regards.
Comment 3 Yousuf Philips (jay) (retired) 2017-07-18 17:02:13 UTC
(In reply to Alex ARNAUD from comment #1)
> ... but it's more logical to read the word in most language
> from left to right so I prefer to have the cursor at the beginning

Having the cursor at the end makes it easier for users to modify it and more people make errors at the end of a word than at the beginning. This is also the same behaviour available in MSO.
Comment 4 Michael Weghorn 2023-11-02 18:22:35 UTC
In current LO versions, the cursor is already at the *beginning* of the misspelled word when the text field gets focus.

Bibisecting shows that this was implemented in this commit:

commit 243b5b392906042ab03800e0b5765e6f3513372c
Author: Caolán McNamara
Date:   Fri Jun 14 21:56:44 2019 +0100

    weld SpellDialog
    
    a) use EditEngine instead of TextEngine as the former can be hosted in a
       foreign widget
    b) use a SfxGrabBagItem to hold the custom spellchecking info inside the
       EditEngine
    c) in longer paragraphs the current word is now auto-scrolled into view
    d) rename Invalidate to InvalidateDialog
    
    Change-Id: Ic6db019c32cdfd5f354c58ee7394fdaa040b86e1
    Reviewed-on: https://gerrit.libreoffice.org/74535
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara
    Tested-by: Caolán McNamara

-> closing as fixed (It's the beginning of the word, not the end, but even in the discussion in this ticket, there were different opinions, so I think that should be OK.)

(In reply to Yousuf Philips (jay) (retired) from comment #0)
> When the spelling dialog is opened and when a new mispelt word is
> highlighted, the cursor should return to the 'not in dictionary' text area
> so that a user can easily modify the word and a screen reader can read the
> mispelt word, as presently a a11y users cant know which of the text area
> words are mispelt.

Not sure whether the cursor should automatically return to the text field, since other use cases might prefer e.g. having one of the buttons focused by default so that e.g. correcting/ignoring/adding words using the keyboard is faster.
In any case, Alt+N can be used to quickly switch focus to the text edit with the misspelled word.
(I can't find it now, but I think there might even have been another bug report requesting that focus should be on a certain button.)


Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 142e7d985f2f6cdb1e37dc0540d77e1fbd6f1bfd
CPU threads: 12; OS: Linux 6.5; UI render: default; VCL: gtk3
Locale: en-GB (en_GB.UTF-8); UI: en-US
Calc: threaded