[Libreoffice-bugs] [Bug 139967] "Ignore double spaces" option in AutoCorrect might be better labeled "prevent double spaces" - and behaves differently than Help page

[bugzilla-daemon at bugs.documentfoundation.org]

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

--- Comment #2 from sdc.blanco at youmail.dk ---
From:
https://opengrok.libreoffice.org/xref/core/officecfg/registry/schema/org/openoffice/Office/Common.xcs?r=35713de9#1374

<prop oor:name="RemoveDoubleSpaces" oor:type="xs:boolean" oor:nillable="false">
  <!-- OldPath: AutoCorrect/Options/All -->
  <!-- OldLocation: Soffice.cfg -->
  <!-- UIHints: Tools  AutoCorrect/AutoFormat  Options  Ignore Double
       Spaces -->
  <info>
    <desc>Specifies if multiple spaces should be combined into one.</desc>
    <label>Ignore double spaces</label>
  </info>
 <value>false</value>
</prop>

Was the intention to create a "removedoublespaces" function, and along the way,
the vision changed to "prevent" double spaces, without a corresponding updating
here in the registry  (which then got copied into Help?) 

If there is no intention for a "remove double spaces" function, then a friendly
source code cleanup would be to rename "RemoveDoubleSpaces" to
"PreventDoubleSpaces" and rename the "IgnoreDoubleSpace" flag (
include/editeng/svxacorr.hxx ), which is used in (
editeng/source/misc/acorrcfg.cxx  and cui/source/tabpages/autocdlg.cxx ) to
something like "NoDoubleSpace"

cui/inc/strings.hrc  to change label of Option

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20210202/cca4846e/attachment.htm>