Bug 98902 - Editeng: when the context is a selection, show the normal, non-spellchecking context menu
Summary: Editeng: when the context is a selection, show the normal, non-spellchecking ...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium enhancement
Assignee: Justin L
URL:
Whiteboard: target:26.2.0
Keywords: needsDevEval
Depends on:
Blocks: Spell-Checking
  Show dependency treegraph
 
Reported: 2016-03-26 10:05 UTC by jose.velez
Modified: 2025-10-09 12:15 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot MS Word (4.10 KB, image/png)
2024-05-17 07:53 UTC, Heiko Tietze
Details
Impress file with misspelled words (14.81 KB, application/vnd.oasis.opendocument.presentation)
2025-09-28 08:36 UTC, jose.velez
Details

Note You need to log in before you can comment on or make changes to this bug.
Description jose.velez 2016-03-26 10:05:52 UTC
1) Set on Automatic spell checking using English language.
2) Write this text in a slide:

Skdjaf jksdfh the car is red dskfh skdjfh

3) Select the whole text

- If you right click on the "car" word the behavior is as expected (the context menu is showed).
- If you right click on a misspelled word the text is unselected, and only the misspelled word is selected. You can recover the selection if you right click in any place.

In think that this behavior is confusing. You could think, that it is designed to work in this way, but this behavior is different from the one obtained in writer. In writer the selection never change with a right click.

Perhaps this bug is similar to 62745, but I think that it is not the same, because 62745 doesn't address the select/unselect aspect.
Comment 1 Buovjaga 2016-04-04 17:06:39 UTC
Repro.

64-bit, KDE Plasma 5
Build ID: 5.1.1.3 Arch Linux build-2
CPU Threads: 8; OS Version: Linux 4.4; UI Render: default; 
Locale: fi-FI (fi_FI.UTF-8)
Comment 2 QA Administrators 2017-10-23 14:03:05 UTC Comment hidden (obsolete)
Comment 3 jose.velez 2018-02-23 10:42:06 UTC
The bug persists in LibreOffice 6.0.
Comment 4 jose.velez 2018-10-20 06:15:51 UTC
Additionally, you can't copy a misspelled word using the contextual menu because the copy option does not appear.
Comment 5 QA Administrators 2019-10-21 02:29:39 UTC Comment hidden (obsolete)
Comment 6 jose.velez 2020-03-27 21:55:51 UTC
The bug is present in 6.4.2.
Comment 7 Buovjaga 2020-03-28 06:17:04 UTC Comment hidden (obsolete)
Comment 8 Rainer Bielefeld Retired 2022-05-03 17:17:46 UTC
a) Also in AOO4111m1(Build:9808)  -  Rev. bdb20b2a64 2021-09-20 16:18 on WIN 64
So eventually inherited from OOo.

b) Still REPRODUCIBLE with Server Installation of Version: 7.4.0.0.alpha0+ (x64)  Build ID b000d964fcc8849d10576bf3539bde7729db2eb1
CPU threads: 12; OS: Windows 10.0 Build 19044; UI render: default; VCL: win
Locale: de-DE (de_DE); UI: en-US  |  Calc: CL  |  Auto Colibre Theme  |  Special devUserProfile

c) IMHO this one is a DUP of IMPRESS "Bug 62745 - EDITING: right-click inconsistency on selected text"

d) And IMHO this is not necessarily a bug – it's simply a possibility to show up the spellcheck context menu on misspelled words if necessary. 

But indeed, the difference to writer behavior is ugly. And I think behavior should be unified.

e) And BTW: Also CALC behaves same way like DRAW. In calc if you select complete cell with text contents rigtclick on misspelled word will release highlighting of other strings in cell and show up spell check context menu. I can't tell whether we need a separate Bug for CALC.

f) Probably "Bug 147269 - LibreCalc - no menu on the cell right click due to flagged spelling" is concerning the related CALC problem

g) I doubt that "Bug 132629 - can't spell check using right click in merged cells" is related
Comment 9 QA Administrators 2024-05-04 03:17:15 UTC Comment hidden (obsolete)
Comment 10 jose.velez 2024-05-06 11:37:06 UTC
Claro, aquí tienes una versión revisada para mejorar el estilo y la claridad:

The bug persists in LibreOffice Impress 24.2.2.

This issue in Impress stems from LibreOffice Text Boxes.

Similarly, Text Boxes in Writer exhibit the same problem.

The behavior of text boxes and the text editor in Writer lacks consistency.
Comment 11 Rainer Bielefeld Retired 2024-05-06 14:59:17 UTC
Still  REPRODUCIBLE with Server Installation of Version: 24.8.0.0.alpha0+ (X86_64)
Build ID: 2146e66d8df2b7b6a2dd868e886cae76aaf7f48b
CPU threads: 12; OS: Windows 10.0 Build 19045; UI render: default; VCL: win
Locale: Locale: de-DE (de_DE); UI: de-DE; Theme: Colibre
Calc: CL threaded – normal Test Profile created from 7.6

And still: 
d) IMHO this is not necessarily a bug – ...
Comment 12 Heiko Tietze 2024-05-07 07:46:45 UTC
At least it is inconsistent with Writer. Actually I would prefer the Impress-way as it allows to do spellchecking while text is selected.
Comment 13 Eyal Rozenberg 2024-05-14 22:37:24 UTC Comment hidden (no-value)
Comment 14 jose.velez 2024-05-16 09:01:51 UTC
The behavior remains the same. Nothing changes in version 24.2.3.2.

If you select text that contains misspelled words in Writer, right-clicking does not display the spell checker. However, in Impress, it does. Essentially, the behavior differs depending on whether you are in the Writer editor or a text box.

Perhaps, this discrepancy could not be considered a bug, but the inconsistent behavior within the same program is confusing.
Comment 15 Heiko Tietze 2024-05-17 07:53:47 UTC
Created attachment 194163 [details]
Screenshot MS Word

We discussed the topic in the design meeting.

The way Impress handles spelling errors looks very convenient and should be made available in Writer as well. Some blocker may result from the missing capability to deal with phrases (introduced for bug 154499). What do you think, also regarding feasibility and effort László?

As an improvement we should keep the user-selection but overlay it with the spell-check color. Similar to how MS Word handles it, see screenshot.
Comment 16 Justin L 2025-09-27 17:45:43 UTC
I must confess that I don't understand this bug report at all. To me, it seems as though Writer and Impress handle textbox content the same way...

A sample document is a must...
Comment 17 jose.velez 2025-09-28 08:36:45 UTC
Created attachment 203002 [details]
Impress file with misspelled words

In the attached Impress document:

Activate Automatic spell checking.

Double-click on any misspelled word to select it.

Right-click on the selection to open the context menu and try to copy it.

👉 The Copy option does not appear in the context menu.

This means that it is not possible to copy a misspelled word using only the mouse.

In contrast, in Writer (the editor), the Copy option is always available in the context menu, regardless of whether the word is misspelled.
It is true that text boxes inside Writer behave the same as in Impress, but this inconsistency between text boxes and the main Writer editor is confusing for users.

The issue is not only about spellchecking, but also about the unexpected difference in basic functionality (copy/paste) between Writer and Impress.
Comment 18 Justin L 2025-09-29 16:44:19 UTC
OK - so this bug report is about Impress/Draw (editeng really). Marking as Impress again instead of comment 15's change to Writer.

And the request is for the opposite of what comment 15 suggests. Hiding it as off-topic.

In normal writer - the incorrect word is ignored if there is a selection, so the user just gets the normal context menu.  (that is why bug 62745 is marked as WORKSFORME). This bug report requests the same for editeng.

Currently in editeng components (impress/draw/textboxes), selected text still results in a spell-check context menu. "This means that it is not possible to copy a misspelled word using only the mouse." Thus the request here is to do the opposite of the comment 15's title change to "Allow spellchecking within selections". Changing the title again...
Comment 19 Justin L 2025-09-30 00:27:01 UTC
since initial import: For writer, SwView::ExecSpellPopup
    if( pVOpt->IsOnlineSpell() &&
        !m_pWrtShell->IsSelection())
Comment 20 Justin L 2025-09-30 14:40:08 UTC
This change will need some serious testing. One possibility would be that no context menu is shown at all. The other test that should be done is to "cut" from the new context menu - and ensure that the selection is actually cut. An interesting test would be to select only PART of a misspelled word, and note the difference based on where the mouse pointer is.

Launching the context menu via Shift-F10 would also be a good test.

In my testing:
-Calc comments: ALREADY don't have a functional context menu.
-Calc textbox/shapes: ALREADY opens the normal context menu.
-Calc cell text: patch works nicely.

-Draw: textbox/table/shape/fontwork : patch works nicely.
-Draw comments: no spell checking done in them.

-Impress: View - Notes Pane: patch works nicely.
-Impress: View - Outline: patch works nicely.

-Writer textbox / comments: patch works nicely.
Comment 21 Commit Notification 2025-09-30 14:40:53 UTC
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/b1c0c4838d2e006ffa8e72516569ce8d13bdbe01

tdf#98902 editeng: no SpellPopup when IsSelectionAtPoint

It will be available in 26.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.