librehat / com.librehat.yahooweather

[DEPRECATED] Yahoo! Weather Widget for Plasma Desktop
https://www.kde-look.org/p/998912/
GNU General Public License v3.0
23 stars 11 forks source link

Re #62: Tray icon immediately shows current conditions at log-in. #65

Closed smitgd closed 8 years ago

smitgd commented 8 years ago

This also adds the current temperature with the tray icon (with no F or C units to minimize space), and adds a prompt to the tooltip contents to "click on the tray icon for error details" when there is an error detected (e.g., bad woeid). A click on icon will cause the full representation to appear containing the usual error display that is not possible to show in the tray. In addition, "backend" code (Yahoo.qml) is now included at the main.qml level so only 1 instance of data/api code exists and onComplete() only does a query if no query has yet occurred, typically at startup. new file: plasmoid/contents/ui/CompactWx.qml modified: plasmoid/contents/ui/Weather.qml modified: plasmoid/contents/ui/Yahoo.qml modified: plasmoid/contents/ui/main.qml

smitgd commented 8 years ago

I was able to just move the Yahoo {id: backend} reference thing into main.qml and reference objects in views with backend.*. So Yahoo.qml still exists and main.qml remains small. Results in one data instance that is common/shared between the two views.

smitgd commented 8 years ago

Superceded by pull request #66