Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[application][pvr][estuary] Fixes after #13699 #13715

Merged
merged 2 commits into from
Mar 31, 2018

Conversation

ksooo
Copy link
Member

@ksooo ksooo commented Mar 30, 2018

Small fixes after #13699

  1. Show busy dialog when starting playback also for pvr recordings.
  2. Ensure player cache level progress and busy spinner are not shown at the same time (prefer cache level progress), remove animations because switching between progress/busy must be instantly.

@ronie @FernetMenta good to go?

… shown at the same time (prefer cache level progress), remove animations because switching between progress/busy must be instantly.
@ksooo ksooo added Type: Fix non-breaking change which fixes an issue Component: PVR Component: Video Component: Skin labels Mar 30, 2018
@ksooo ksooo added this to the Leia 18.0-alpha2 milestone Mar 30, 2018
@ksooo ksooo requested review from FernetMenta and ronie March 30, 2018 15:35
@ksooo ksooo merged commit bc1e8d4 into xbmc:master Mar 31, 2018
@ksooo ksooo deleted the fix-busydialog-on-playbackstart branch March 31, 2018 19:48
@Rechi Rechi added the v18 Leia label Mar 31, 2018
ksooo added a commit to ksooo/xbmc that referenced this pull request Apr 4, 2018
ksooo added a commit to ksooo/xbmc that referenced this pull request Apr 4, 2018
ksooo added a commit to ksooo/xbmc that referenced this pull request Apr 4, 2018
ksooo added a commit to ksooo/xbmc that referenced this pull request Apr 5, 2018
ksooo added a commit that referenced this pull request Apr 5, 2018
[Estuary] Reintroduce/fix busy dialog animations after #13715.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants