SpicyChair / pluvia_weather_flutter

Pluvia 🌦️ | A weather app with beautiful animations, built with Flutter.
GNU General Public License v3.0
144 stars 15 forks source link

Temperature overlapping city name #25

Closed whats-a-diorama closed 1 year ago

whats-a-diorama commented 2 years ago

Love the app. It's gorgeous, and has all the information that I'm looking for. Thank you for all of the work you've put into it!

I've just had one consistent minor cosmetic issue that I thought I'd report. The temperature display is always overlapping the city name on my device. So it never quite lines up with how it looks in the screenshots. I attached a screenshot of my own as an example. Sometimes it looks a little worse, depending on the numbers.

I haven't noticed anything like this anywhere else in the app.

F-Droid build of Pluvia, version 1.4.5. Pixel 5a (2400x1080 resolution), Android 11.

pluvia-temperature

SpicyChair commented 2 years ago

Thank you for your support!

That's an interesting bug I haven't seen before - what is your DPI/Scaling?

whats-a-diorama commented 2 years ago

My "Display size" configuration is set to "Default", which appears to be a DPI of 420. I do have my "Font size" configuration set to "Small", which is one step below "Default". But changing it back to "Default" doesn't appear to resolve the issue, unfortunately. Everything will be a little bigger, but still positioned identically.

whats-a-diorama commented 2 years ago

Hi. Just wanted to add that I've since upgraded to Android 12, but unfortunately that didn't have any impact on the issue. It still looks exactly the same.

(Though outside of that, the app is working great for me in Android 12!)

ghost commented 2 years ago

Greetings, want to thank you for creating this cool app. Writing this message here because it could be relative to this issue and if it's an issue at all. I'm using crDroid ROM for mido (Android 11), 457 DPI. App version 1.4.7 from F-Droid. Screenshot_20220103-093839909_1

n8x0b7T commented 1 year ago

Instead of the city name being in the top bar, the city name text and refresh button should be in the same container as the temperature text. This would eliminate all the above issues.

SpicyChair commented 1 year ago

@n8x0b7T Good spot, thanks - I'll add this today

SpicyChair commented 1 year ago

Fixed with latest release - will be on Github soon