Review Request: Make condition icons work in NOAA when conditions are translated

Lasse Liehu lliehu at kolumbus.fi
Mon Feb 7 02:01:35 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100589/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

The condition icons didn't always show up correctly because i18n'd condition strings were passed to getConditionIcon(). This tries to fix it, passing it the original strings.


Diffs
-----

  plasma/generic/dataengines/weather/ions/noaa/ion_noaa.h 84e8d6f 
  plasma/generic/dataengines/weather/ions/noaa/ion_noaa.cpp 4987968 

Diff: http://git.reviewboard.kde.org/r/100589/diff


Testing
-------

The correct icons are used on the summary (tested during both day and night) and forecasts, using the "weather" applet and Finnish. The conditions in forecasts' tooltips are still translated.


Thanks,

Lasse

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110207/1bfd0d05/attachment.htm 


More information about the Plasma-devel mailing list