[Bug 160834] Unclear meaning of "Ignore pieces of length"

2024-05-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=160834

Heiko Tietze  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Assignee|libreoffice-b...@lists.free |heiko.tietze@documentfounda
   |desktop.org |tion.org
 Ever confirmed|0   |1
 CC|libreoffice-ux-advise@lists |heiko.tietze@documentfounda
   |.freedesktop.org|tion.org
   Keywords|needsDevAdvice, needsUXEval |

--- Comment #5 from Heiko Tietze  ---
Presented the case in the ESC and got acceptance to remove the UI access.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 160834] Unclear meaning of "Ignore pieces of length"

2024-05-13 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=160834

--- Comment #4 from Heiko Tietze  ---
Testing with the autotext of lorem, I see a difference in the track changes
depending on word vs. character by how the modification is marked. I don't see
any effect in the piece length, eg. ibzum vs. ipsum is always marked as either
the word is completely different or the particular characters regardless a
piece length of 1 or 3.

We may just hide the option to solve the UI issue.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 160834] Unclear meaning of "Ignore pieces of length"

2024-05-13 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=160834

--- Comment #3 from Noel Grandin  ---
Given the source file, it has something to do with comparing documents, other
than that, I don't know.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 160834] Unclear meaning of "Ignore pieces of length"

2024-05-13 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=160834

Heiko Tietze  changed:

   What|Removed |Added

   Keywords||needsDevAdvice
 CC||noelgran...@gmail.com

--- Comment #2 from Heiko Tietze  ---
The property IgnorePieces in
officecfg/registry/schema/org/openoffice/Office/Writer.xcs has the description
"Specifies whether short pieces of matched text are ignored." 

IsIgnorePieces() aka m_bIgnorePieces defined in sw/inc/modcfg.hxx is used as
CmpOptions.nIgnoreLen = SW_MOD()->IsIgnorePieces() ? SW_MOD()->GetPieceLen() :
0; in sw/source/core/doc/doccomp.cxx ultimately applied in
CommonSubseq::IgnoreIsolatedPieces(). Quickly skimming over the code it seems
the option controls whether "isolated pieces" (characters) are ignored and up
to which length.

I guess the wording "pieces" is used because the option is not only applied in
case of (compare) "By Character" but also "By word".

Noel, you made a lot of changes in the source code. Please enlighten us.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 160834] Unclear meaning of "Ignore pieces of length"

2024-05-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=160834

QA Administrators  changed:

   What|Removed |Added

 Whiteboard| QA:needsComment|

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 160834] Unclear meaning of "Ignore pieces of length"

2024-05-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=160834

Dieter  changed:

   What|Removed |Added

 CC||dgp-m...@gmx.de,
   ||libreoffice-ux-advise@lists
   ||.freedesktop.org
   Keywords||needsUXEval
 Blocks||125674

--- Comment #1 from Dieter  ---
(In reply to Tuomas Hietala from comment #0)
> Description:
> The UI string "Ignore _pieces of length:" is somewhat confusing and
> contradictory. The following explanation is given in Help: "Set the minimum
> number of characters to trigger a valid comparison."
> 
> The UI and Help are at odds here, because it's not clear whether the
> selected number of characters is to be ignored (as indicated by the UI
> string) or if it's the minimum amount to be taken into account (as indicated
> by Help)

I agree and couldn't really figure out, what is the current behaviour
=> NEW


> "Pieces of length" should IMO also be reworded for clarity. I'd suggest
> replacing the string with something like "Minimum number of characters to
> compare" (if Help is correct) or "Maximum number of characters to ignore"
> (if Help is not).
I support a terminology, that is much clearer, but I don't know, what's the
best solution here
cc: Design-Team (who should also decide, if we should separate the two issues
into two different reports.


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=125674
[Bug 125674] [META] Document Comparison bugs and enhancements
-- 
You are receiving this mail because:
You are on the CC list for the bug.