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

loading spinner, Progress Dialog and movie details button animate #827

Merged

Conversation

candry7731
Copy link
Contributor

Added loading spinner to movies detail screens

Added loading spinner to the movie details screen

Added progress dialog when user clicks the play button for movies and trailers

Updated movie details buttons to be hidden until all data is loaded

Updated critic rating star to be hidden until all data is loaded

Added Loading spinner to movies detail screens
@jimdogx
Copy link
Contributor

jimdogx commented Nov 5, 2022

I noticed that when I hit Play Movie, I get the new "Loading Movie" spinner, that goes away and then I get the regular spinner (not text). Not sure if that was the desired behavior?

@whiteowl3
Copy link
Contributor

If the server holding the strm fails to respond, the spinner locks us out forever. back or instant replay should cancel the loading job and clear the spinner.
Why it is so visually different from all the other spinners?

@whiteowl3
Copy link
Contributor

whiteowl3 commented Nov 6, 2022

I noticed that when I hit Play Movie, I get the new "Loading Movie" spinner, that goes away and then I get the regular spinner (not text). Not sure if that was the desired behavior?

The first spinner is to lock out additional inputs (esp play inputs) until jellyfin responds with the stream. if the media is remote, this can take a second. so, the first spinner drops as soon as jellyfin says "here it is" and the second spinner waits until we have enough buffered to play. we need both.

@github-actions
Copy link

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@github-actions github-actions bot added the stale This issue/PR has gone stale. label Nov 28, 2022
@candry7731
Copy link
Contributor Author

This should be good to go unless anyone else finds any bugs.

@github-actions github-actions bot removed the stale This issue/PR has gone stale. label Nov 29, 2022
@1hitsong
Copy link
Member

This works as described for me.

@jimdogx You good?

@candry7731 Can you resolve the merge conflict?

@jimdogx
Copy link
Contributor

jimdogx commented Dec 23, 2022

This might be due to the conflicts, but if I run this branch on a movie that is partially watched (so I should see the "Resume" or "start over" popup), I am shown "Loading Movie" with a spinner that never goes away.

@github-actions
Copy link

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@github-actions github-actions bot added the stale This issue/PR has gone stale. label Jan 14, 2023
@1hitsong 1hitsong removed the stale This issue/PR has gone stale. label Jan 20, 2023
@1hitsong
Copy link
Member

1hitsong commented Feb 1, 2023

Status: Pending code updates.

@candry7731
Copy link
Contributor Author

This might be due to the conflicts, but if I run this branch on a movie that is partially watched (so I should see the "Resume" or "start over" popup), I am shown "Loading Movie" with a spinner that never goes away.

This should be fixed. Please test and let me know.

@sevenrats
Copy link
Member

it's important that we restart the progress dialog in VideoPlayer.brs after we have closed it in startPlayBackOver.

@1hitsong
Copy link
Member

1hitsong commented Feb 10, 2023

From Home View, Click TV Episode from Continue Watching

  • From popup click Go To Series. Spinner shows and doesn't go away
  • From popup click Go To Season. Spinner shows and doesn't go away
  • From popup click Go To Episode. Spinner shows and doesn't go away

@candry7731
Copy link
Contributor Author

I'm also getting an error when clicking on the season...

image

@candry7731
Copy link
Contributor Author

I'm working on these now.... Just fyi

@candry7731
Copy link
Contributor Author

These are completed and ready for review

Copy link
Member

@1hitsong 1hitsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionality looks good. Just a few small finishing touches and it's done.

components/movies/MovieDetails.xml Outdated Show resolved Hide resolved
components/movies/MovieDetails.xml Outdated Show resolved Hide resolved
source/Main.brs Show resolved Hide resolved
Copy link
Member

@1hitsong 1hitsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be merged 2/12 unless an issue arises.

@1hitsong 1hitsong assigned 1hitsong and unassigned candry7731 Feb 11, 2023
@1hitsong 1hitsong merged commit 2cf133e into jellyfin:unstable Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants