After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 138106 - The "Error" message shouldn't have thousand separation.
The "Error" message shouldn't have thousand separation.
Status: RESOLVED FIXED
Product: gnome-calculator
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Rich Burridge
Rich Burridge
Depends on:
Blocks:
 
 
Reported: 2004-03-25 16:02 UTC by Rich Burridge
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix for the problem. (525 bytes, text/plain)
2004-03-28 23:56 UTC, Rich Burridge 		Details
Hopefully a real fix for the problem. (1.40 KB, text/plain)
2004-06-09 18:15 UTC, Rich Burridge 		Details

Description Rich Burridge 2004-03-25 16:02:18 UTC 
To repeat:

* Start GNOME 2.5 gcalctool and turn on "Show Thousands Separator".
* View Scientific mode and enter "12345"
* Click on the x! (factorial) button.

The display will show: "Er,ror".
Comment 1 Rich Burridge 2004-03-28 23:56:11 UTC 
Created attachment 26039 [details]
Fix for the problem.
Comment 2 Rich Burridge 2004-03-29 00:01:02 UTC 
Changes applied to CVS HEAD. The version number in
configure.in has been bumped to 4.4.3.
Comment 3 Raj 2004-06-08 04:15:46 UTC 
The problem is fixed in Scientific mode, but it still exists in Basic/Financial
mode. Verified that the problem exists.

Steps to reproduce the problem:
1. Launch Calculator and set it in basic mode.
2. Select View->Show Thousands Separator
3. Click 'Clr'.
4. Click '1/x'

The message 'Error' is displayed as 'Err,or'. Please reopen.
Comment 4 Rich Burridge 2004-06-09 18:15:54 UTC 
Created attachment 28519 [details]
Hopefully a real fix for the problem.
Comment 5 Rich Burridge 2004-06-09 18:20:34 UTC 
Well, the first patch is part of the solution, but v->error
needs to be set non-zero for "Error" to be correctly displayed.
The second patch provides this. 

Changes applied to CVS HEAD. Version number in configure.in
bumped to 4.4.11