Long forecast text cut off #57

Closed
opened 11 months ago by hirnsushi · 11 comments

3 digit percentages or longer forecast text lead to the last line being cut off.
I'm using Tiny Weather Forecast on a Nexus 4, so people with "normal" sized phones probably haven't noticed yet 😏
Edit: Weather warnings and sunrise / sunset information also clash sometimes.

3 digit percentages or longer forecast text lead to the last line being cut off. I'm using Tiny Weather Forecast on a Nexus 4, so people with "normal" sized phones probably haven't noticed yet 😏 Edit: Weather warnings and sunrise / sunset information also clash sometimes.
Owner

This should be fixed now: 00929ea8fa

  1. Devices with the smallest screen diameter being below 360dp (this should be e.g. the Nexus 4) now get an alternative layout of the weather items in the main app, mainly consisting of a little bit smaller text sizes. This should also correct the wrong-positioned weather warning icon, that is currently being partly overlapped by sunset/sunrise in your picture.

  2. Each listview item (means e.g. the 6h display) now gets adapted to the real size required by the items after the layout gets inflated. In reality, this means that the height should get bigger if needed, guaranteeing that all items fit within.

  3. The probabilities on the left side now never line-break, in the worst case they get cut-off. However, this should not happen anymore due to improvement #1. :-)

If you like, you can test the latest unserdebug from today (download here) and give feedback if it now looks ok.

This should be fixed now: https://codeberg.org/Starfish/TinyWeatherForecastGermany/commit/00929ea8fad7f79ea29b28d80f54bd711c0858b7 1. Devices with the smallest screen diameter being below 360dp (this should be e.g. the Nexus 4) now get an alternative layout of the weather items in the main app, mainly consisting of a little bit smaller text sizes. This should also correct the wrong-positioned weather warning icon, that is currently being partly overlapped by sunset/sunrise in your picture. 2. Each listview item (means e.g. the 6h display) now gets adapted to the real size required by the items *after* the layout gets inflated. In reality, this means that the height should get bigger if needed, guaranteeing that all items fit within. 3. The probabilities on the left side now never line-break, in the worst case they get cut-off. However, this should not happen anymore due to improvement #1. :-) If you like, you can test the latest unserdebug from today ([download here](https://kaffeemitkoffein.de/nextcloud/index.php/s/4SXHaLxLSmFd8Ri)) and give feedback if it now looks ok.
Poster

That was fast, thank you 🙂
I tried TinyWeatherForecastGermany-build021-version_0.55.1-userdebug-20210313-01.apk, should I wait for a newer build, still getting cut off text 🤔

That was fast, thank you 🙂 I tried TinyWeatherForecastGermany-build021-version_0.55.1-userdebug-20210313-01.apk, should I wait for a newer build, still getting cut off text 🤔
Owner

Not good - please test the userdebugs named Nexus4 1-5 in this order and tell me if any of them works like expected and if the background gets dark-green.

The color is only to get sure you get the alternative layout for narrow devices applied.

Not good - please test the userdebugs named Nexus4 1-5 in this order and tell me if any of them works like expected and if the background gets dark-green. The color is only to get sure you get the alternative layout for narrow devices applied.
Poster
  1. Version
    • Can't spot a difference
    • Normal background
  2. Version
    • Text fits
    • Weather warning still overlapping
    • List item grows endlessly when scrolling up / down (screenshot)
    • Normal background
  3. Version
    • Same as 2.
  4. Version
    • Same as 2.
  5. Version
    • Same as 2.

Sorry, I don't seem to be able to trigger the alternative layout… 🤔
I tried different font sizes in the Android display settings. That gets rid of the overlapping weather warning if set to standard (hardly usable on that display size), still no green background.

1. Version - Can't spot a difference - Normal background 2. Version - Text fits - Weather warning still overlapping - List item grows endlessly when scrolling up / down (screenshot) - Normal background 3. Version - Same as 2. 4. Version - Same as 2. 5. Version - Same as 2. Sorry, I don't seem to be able to trigger the alternative layout… 🤔 I tried different font sizes in the Android display settings. That gets rid of the overlapping weather warning if set to standard (hardly usable on that display size), still no green background.
Owner

ok, thanks, I realized yesterday late at night that this is buggy :-((, but #2 saying "text fits" already helps a lot :-)

Thanks, I will go on.

ok, thanks, I realized yesterday late at night that this is buggy :-((, but #2 saying "text fits" already helps a lot :-) Thanks, I will go on.
Owner

Sorry for bothering you, would you do a last test?

There is a #6 userdebug, it should now do the trick. (I deleted all the buggy other ones.) The background should now go green unless your device has more than 400dp width, which is unlikely for the Nexus 4.

Can you please also go to "About...", scroll down and give me a screenshot of your screen resolution details?

(You get get the same from the logs, however you need to clear old logs first.)

This would help a lot to be sure how many dp your device (the Nexus 4) really has availabe and will help me to determine the point more exactly where the cut-off is to change layouts.

Sorry for bothering you, would you do a last test? There is a #6 userdebug, it should now do the trick. (I deleted all the buggy other ones.) The background should now go green unless your device has *more than 400dp width*, which is unlikely for the Nexus 4. Can you please also go to "About...", scroll down and give me a screenshot of your screen resolution details? (You get get the same from the logs, however you need to clear old logs first.) This would help a lot to be sure how many dp your device (the Nexus 4) *really* has availabe and will help me to determine the point more exactly where the cut-off is to change layouts.
Owner

Should be fixed now: some items are now smaller if the screen width is below 400dp or 320dp. Furthermore, there were multiple tiny bugs that taken together made it possible for the forecast bar to overlap the text when warnings were in a closed state (not visible), what was never intended. Should work now. At least, it does with an older device (Samsung Galaxy S3) that I have.

Should be fixed now: some items are now smaller if the screen width is below 400dp or 320dp. Furthermore, there were multiple tiny bugs that taken together made it possible for the forecast bar to overlap the text when warnings were in a closed state (not visible), what was never intended. Should work now. At least, it does with an older device (Samsung Galaxy S3) that I have.
Owner

Fixed: b0e46002dc

Fixed: https://codeberg.org/Starfish/TinyWeatherForecastGermany/commit/b0e46002dc06d426936b7823f7bc119c4babc4df
Poster

Green background, fitting text and no overlapping weather warning, success 🤘
Thank you so much for fixing this, would have stayed a user regardless, but now it's way more convenient 🙂
Let me know if you need any more information or logs.

Green background, fitting text and no overlapping weather warning, success 🤘 Thank you so much for fixing this, would have stayed a user regardless, but now it's way more convenient 🙂 Let me know if you need any more information or logs.
Starfish added the
bug
label 11 months ago
Owner

I am happy that it works now! I get a lot of feedback/bug reports for quite old devices. From an ecological perspective, I am happy that not everyone is throwing away their devices after two years, so I am also happy to support them.

I am happy that it works now! I get a lot of feedback/bug reports for quite old devices. From an ecological perspective, I am happy that not everyone is throwing away their devices after two years, so I am also happy to support them.
Starfish closed this issue 11 months ago
Poster

That's exactly my reasoning. This phone hasen't let me down in 8 years while still running the latest security patches and Android 10 (almost 11). Why change 🤷 😏
Enough off topic, thx again 🙂

That's exactly my reasoning. This phone hasen't let me down in 8 years while still running the latest security patches and Android 10 (almost 11). Why change 🤷 😏 Enough off topic, thx again 🙂
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.