https://bugs.documentfoundation.org/show_bug.cgi?id=81894

Justin L <jl...@mail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|libreoffice-b...@lists.free |jl...@mail.com
                   |desktop.org                 |
           See Also|                            |https://bugs.documentfounda
                   |                            |tion.org/show_bug.cgi?id=74
                   |                            |087
             Status|NEW                         |ASSIGNED

--- Comment #33 from Justin L <jl...@mail.com> ---
Focusing on comment 6's Attachment 104541 and cell B12, I bisected that to
bibisect-linux-43max commit de7e6a479d5af64e330bcbac2bc491e79076039b

source commit cd22c3442389f69fc1cc14dd07b17f5a59498e5e
Author: Kohei Yoshida on Fri Apr 11 23:09:38 2014 -0400
        fdo#74087: Inspect if a mis-spelled word is at cursor position...
        then decide whether to launch a spell candidate menu or a regular
        context menu.

https://cgit.freedesktop.org/libreoffice/core/log/?id=47a77d7dbc427e51421e2df8d59695834cb74980&qt=range&q=cd22c3442389f69fc1cc14dd07b17f5a59498e5e

This was the commit that introduced IsSpellErrorAtPos(), and it was a follow-up
fix for the first one mentioned in comment 29. It both allowed spell checking
of spill-out text, and allowed IsSpellErrorAtPos to cancel a bSpellError.

But it is fairly simple to return bDone=false from ExecuteSpellPopup if there
is no misspelled word at that PixelPos - and that is much more accurate. If not
done, then the  non-spelling popup will display (but care needs to be taken to
close editing first if necessary).

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to