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

Migrate to fragments and improvements #499

Merged
merged 1 commit into from
Apr 11, 2017

Conversation

mauriciocolli
Copy link
Contributor

  • Migrate to fragments
  • Fix Missing first "related video" in chronological series #487
  • Don't show "Open in popup mode" to channel links
  • New backstack of videos
  • Change the subscribers count to format using NumberFormat, for example some locales use .  others , this handles it automatically (the old method had a bug for leading zero, e.g. 4.82.125 instead of 4.082.125)
  • Add string 'subscribers' for channels with more than 1 subscriber (plural)
  • Popup player chooses the default format and resolution based on the new preference (format)
  • Fix taskaffinity of the router activities
  • Show title before loading, as it is available from the items already loaded

- Migrate to fragments
- Fix TeamNewPipe#487
- Don't show "Open in popup mode" to channel links
- New backstack of videos
- Change the subscribers count to format using `NumberFormat`, for example some locales use `.`  others `,`, this handles it automatically (and the old method had a bug for leading zero, e.g. 4.82.125 instead of 4.082.125)
- Add string 'subscribers' for channels with more than 1 subscriber (plural)
- Popup player chooses the default format and resolution based on the new preference (format)
- Fix taskaffinity of the router activities
- Show title before loading, as it is available from the items already loaded
@mauriciocolli
Copy link
Contributor Author

@theScrabi Hey, I see that you forgot to remove this If in the NewPipeExtractor

@theScrabi
Copy link
Member

theScrabi commented Apr 11, 2017

Holy crap I have the suspicion that after you are done I'm not familiar with my software anymore xD.
However I like that you clean up my crappy work 👍

@theScrabi theScrabi merged commit 746c2a1 into TeamNewPipe:master Apr 11, 2017
@mauriciocolli mauriciocolli deleted the fix-next-video branch April 26, 2017 13:59
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.

2 participants