Bug 92628 - EDITING: Double click in formula sometimes gets wrong word boundaries
Summary: EDITING: Double click in formula sometimes gets wrong word boundaries
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: low minor
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Cell-Formula
  Show dependency treegraph
 
Reported: 2015-07-08 16:24 UTC by Eugene Mah
Modified: 2024-11-19 17:31 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Speadsheet with example formula for testing double click word boundaries (10.80 KB, application/vnd.oasis.opendocument.spreadsheet)
2015-07-08 16:24 UTC, Eugene Mah
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Mah 2015-07-08 16:24:46 UTC
Created attachment 117132 [details]
Speadsheet with example formula for testing double click word boundaries

When double clicking a cell reference inside a formula, sometimes the wrong word boundaries are used resulting in more being selected than expected. Specifically, if the cell reference is followed by numbers, the cell reference and the numbers end up being selected, instead of just the cell reference.

In the spreadsheet attached, the formula

=IF(A3="","",IF(LEN(A3)<=135,A3,IF(LEN(A3)<=260,LEFT(A3,SEARCH(" ",A3,125)),LEFT(A3,SEARCH(" ",A3,130)))))

is used as an example. Double clicking the cell reference A3 works as expected in most cases.

Double clicking the A3 cell reference in the SEARCH() functions results in "A3,125" or "A3,130" being selected instead of just A3 being selected as would be expected.

As a work around, adding a space after the comma (i.e. SEARCH(" ",A3, 130)) makes double clicking work as expected.

Tested using LibreOffice Calc 4.4.3 (Windows) and 4.3.7.2 (Fedora 21)
Comment 1 Joel Madero 2015-07-12 17:20:26 UTC
I'm asking UX for input on this one.

FWIW - same behavior in 3.3 so setting the version to "inherited from OOo"

Also setting to:
Minor - slows down but does not prevent high quality work;
Low - default for minor bugs


@UX - this seems like a user preference issue and I can imagine others disagreeing. I'll leave it in your capable hands to determine if we'll implement this or if we'll close it as WONTFIX.


Thanks for reporting!
Comment 2 Robinson Tryon (qubit) 2016-08-25 05:39:11 UTC Comment hidden (obsolete)
Comment 3 Heiko Tietze 2017-04-26 10:40:13 UTC
Double-clicking SEARCH, IF, LEN selects this word, "A3;125" doesn't. 

Selection works within punctuation for 
=Foo(A1;ABC)
=Foo(A1;B1)
=Foo(123;A1)

but not with
=Foo(A1,123)
=Foo(A1:123)
=Foo(A1;123)
(when A1 is recognized as reference to a cell)

Don't see any reason for this inconsistency. So please fix it.
Comment 4 QA Administrators 2018-04-27 02:30:41 UTC Comment hidden (obsolete)
Comment 5 Eugene Mah 2018-04-27 09:36:36 UTC
Bug still exists in 6.0.3.2 (Fedora 28 beta)

Version: 6.0.3.2
Build ID: 6.0.3.2-4.fc28
CPU threads: 8; OS: Linux 4.16; UI render: default; VCL: gtk3; 
Locale: en-US (en_US.UTF-8); Calc: group
Comment 6 QA Administrators 2019-04-28 02:46:15 UTC Comment hidden (obsolete)
Comment 7 Eugene Mah 2019-04-28 11:31:44 UTC
Bug still exists in 6.2.2.2 (Fedora 30 beta)

Version: 6.2.2.2
Build ID: 6.2.2.2-4.fc30
CPU threads: 4; OS: Linux 5.0; UI render: default; VCL: gtk3; 
Locale: en-US (en_US.UTF-8); UI-Language: en-US
Calc: threaded
Comment 8 QA Administrators 2021-04-28 03:51:14 UTC Comment hidden (obsolete)
Comment 9 Eugene Mah 2021-04-28 23:50:44 UTC
Bug still exists in 7.0.52 (Fedora 33)

Version: 7.0.5.2
Build ID: 00(Build:2)
CPU threads: 12; OS: Linux 5.11; UI render: default; VCL: kf5
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 10 QA Administrators 2023-04-29 03:29:38 UTC Comment hidden (obsolete)
Comment 11 Eugene Mah 2023-04-29 11:02:16 UTC
Bug still exists in 7.5.2.2 (Fedora 38)

Version: 7.5.2.2 (X86_64)
Build ID: 50(Build:2)
CPU threads: 12; OS: Linux 6.2; UI render: default; VCL: kf5 (cairo+wayland)
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 12 Andreas Heinisch 2024-11-19 17:28:17 UTC
Something to check in the depths of the BreakIterator [1]. Imho, we need just to update the break iterator rules.

[1] https://opengrok.libreoffice.org/xref/core/editeng/source/editeng/impedit2.cxx?r=cf42d70e#1580