Bug 71652 - UI: Parameter use description
Summary: UI: Parameter use description
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Spreadsheet (show other bugs)
Version: 4.1.3.2 release
Hardware: All All
: lowest trivial
Assignee: Joel Madero
QA Contact:
URL:
Whiteboard: BSA target:4.2.0
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-15 16:03 UTC by John Smith
Modified: 2014-04-01 00:35 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
A screenshot showing the text. (8.18 KB, image/png)
2013-11-15 16:03 UTC, John Smith
Details

Description John Smith 2013-11-15 16:03:31 UTC
Created attachment 89276 [details]
A screenshot showing the text.

Problem description: 
The parameter description for 'No thousands separator' in the FIXED function is overly complicated and confusing.
Steps to reproduce:
1. Open an instance of Calc, launch the Function Wizard and double-click on the FIXED function.
2. Click in the 'No thousands separator' input box to bring upt the explanation in the wizard.


Current behavior:The current text is displayed, quite correctly.

Expected behavior:
To simplify this text and explicitly explain the point of the parameter, alter the text to something like;
'Thousands separator. If 0 or omitted the locale group separator is used else the separator is suppressed.'


I appreciate this is not a defect as such but the current text is very poor. I don't know of any other way to bring this up than as a bug.
              
Operating System: Windows XP
Version: 4.1.3.2 release
Comment 1 Joel Madero 2013-11-15 18:39:43 UTC
Do you mean replacing:

No thousands separator. True value, if existing and TRUE (unequal to 0), no thousands separators are set.

with 

Thousands separator. If 0 or omitted the locale group separator is used else the separator is suppressed.


??
Comment 2 Joel Madero 2013-11-15 19:02:20 UTC
Assuming that's the case, submitting patch now
Comment 3 Joel Madero 2013-11-15 19:10:35 UTC
Patch submitted - upon review 4.2 should show the new text
Comment 4 John Smith 2013-11-15 22:50:59 UTC
(In reply to comment #1)
> Do you mean replacing:
> 
> No thousands separator. True value, if existing and TRUE (unequal to 0), no
> thousands separators are set.
> 
> with 
> 
> Thousands separator. If 0 or omitted the locale group separator is used else
> the separator is suppressed.
> 
> 
> ??

Yes that's correct. Thank you for prompt attention.
Comment 5 Commit Notification 2013-11-17 06:56:09 UTC
Joel Madero committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=42463328f5803d7d34a3bb05fbe1e24cdc02dff8

fdo#71652 - change thousand separate description



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.