mediathekview / zapp

German public broadcasting live streams as an Android app
https://mediathekview.de/news/zapp/
MIT License
199 stars 33 forks source link

Popup windows don't respect system animation speed on android 8 #374

Closed SebiderSushi closed 1 year ago

SebiderSushi commented 1 year ago

Problem summary

Popup windows don't respect the window animation speed system setting, at least on android 8.1.0

Reproduction environment

App version: 8.3.1 from F-Droid Android version: LineageOS 15.1 (android 8.1.0)

Steps to reproduce

  1. Change window animation speed
    • [On any android version] Enable development settings and change the setting "Window animation scale"
    • [On android 11] Go to Settings > Accessibility and turn on "Remove animations"
  2. Install Zapp 8.2.0
  3. Open Zapp
  4. Go to Settings
  5. Tap any multiple choice setting (e.g. Theme, Start page, Language) or go to "Channel order" and click the question mark icon in the title bar
  6. On the "Media Center" tab, open any entry, then click "Download" or "Share"
  7. While playing a live stream, tap on the "info" or "sleep timer" icon in the top right corner
  8. Upgrade to Zapp 8.3.1 and repeat

Expected behavior

The window opens with the appriopriate animation speed as set in system settings.

Observed behavior

Those were all versions i tested. I don't know which android version is the precise cut-off for this problem.

cemrich commented 1 year ago

Thank you very much for your feedback!

I cannot reproduce this behaviour on Stock Android 10 or 13. Stock Android 8.0 does not provide this system setting.

However, Zapp does not use any fancy components - just MaterialAlertDialogBuilder, which should respect all system settings by default. I supsect this to be a bug in Androids material library. Or, it could be a some custom implementation of early lineage versions.

I don't think this issue is something that can be fixed inside Zapp, I am sorry!