Bug 255086 - Units for Spectra are nm, should be Angstroms
Summary: Units for Spectra are nm, should be Angstroms
Status: RESOLVED FIXED
Alias: None
Product: kalzium
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Kalzium Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-24 06:37 UTC by Michael_Courtney
Modified: 2010-11-20 20:31 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Angstrom patch (4.88 KB, patch)
2010-10-24 13:57 UTC, Etienne
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael_Courtney 2010-10-24 06:37:25 UTC
Version:           unspecified (using KDE 4.5.1) 
OS:                Linux

Checked Lithium, Sodium, and Neon.  All have spectra in nm, but should be Angstroms for numbers reported.

Reproducible: Always

Steps to Reproduce:
Click on spectrum for Lithium, reports 6707.77 nm.  It should be 670.777 nm or 6707.77 Angstroms.  Etc.
Comment 1 Alexander Potashev 2010-10-24 13:43:40 UTC
Fixed in http://websvn.kde.org/?view=revision&revision=1132474

How about backporting the fix to 4.5?
Comment 2 Etienne 2010-10-24 13:57:37 UTC
Created attachment 52825 [details]
Angstrom patch

The branch can be patched with that patch. I don't know if i can do that.
Comment 3 Alexander Potashev 2010-10-24 14:47:12 UTC
SVN commit 1189277 by aspotashev:

Backport fix for 255086 to 4.5

CCBUG:255086


 M  +10 -17    spectrumview.ui  
 M  +1 -1      spectrumviewimpl.cpp  
 M  +3 -0      spectrumwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1189277
Comment 4 Etienne 2010-11-20 20:31:07 UTC
See comment #3.
So i close this bug.