search

piggz / harbour-amazfish

Amazfit Bip support for SailfishOS
GNU General Public License v3.0
103 stars 51 forks source link

Improve weather ux #282

Closed jmlich closed 11 months ago

jmlich commented 11 months ago

I wasn't sure which city is currently configured in the layout. I have extended the dialog to display the current city name and current temperature, as shown in the screenshot below:

Screenshot from 2023-10-15 10-28-28

There can also be a prefix such as 'Current city.' Personally, I think it is enough to display just the city name.

LabelPL {
  text: qsTr("Current city: %1 %2")
     .arg(page.currentCity)
     .arg((weather.temperature !== 0) ? " " + (weather.temperature-273) + "˚C" : "")
}

Additionally, an icon can be added and displayed on the same line, for example, using icons from https://erikflowers.github.io/weather-icons/