Bug 255086

Summary:	Units for Spectra are nm, should be Angstroms
Product:	[Applications] kalzium	Reporter:	Michael_Courtney
Component:	general	Assignee:	Kalzium Developers <kalzium>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aspotashev, etienne.rebetez
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Angstrom patch

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.