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

fixup for #2149 (missing buttons) #2487

Merged
merged 5 commits into from
Aug 12, 2019
Merged

fixup for #2149 (missing buttons) #2487

merged 5 commits into from
Aug 12, 2019

Conversation

kapodamy
Copy link
Contributor

@kapodamy kapodamy commented Aug 2, 2019

some buttons on the download page dissapears when the screen goes off or the app is switched

@TobiGr
Copy link
Contributor

TobiGr commented Aug 2, 2019

Can confirm. @theScrabi we might want to add this to 0.17.0, too.

@theScrabi
Copy link
Member

So either @kapodamy you re create this pr but trying to merge into release_0.17.0, or we merge now and rebase into release_0.17.0

@TobiGr
Copy link
Contributor

TobiGr commented Aug 2, 2019

I wanted to rebase the release branch onto dev, but forgot it. Will do it

* fix start/pause buttons disappear, issued by RecyclerView re-draw
* show start/pause buttons in pair to avoid confusions
Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

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

I found another bug. Open the downloads view, switch to another app and back. I'd expect the downloads view to be open, but it is not (mostly the view from which I opened the downloads). When switching apps from another view, the old view is recovered.

app/src/main/res/menu/download_menu.xml Outdated Show resolved Hide resolved
@kapodamy
Copy link
Contributor Author

kapodamy commented Aug 5, 2019

I found another bug. Open the downloads view, switch to another app and back. I'd expect the downloads view to be open, but it is not

@TobiGr
i cannot replicate the bug,¿what do you mean? ¿missing view? ¿the activity itself is closed?

@TobiGr
Copy link
Contributor

TobiGr commented Aug 6, 2019

@kapodamy I also can not reproduce it. I have no idea what caused this strange behavior yesterday.

@theScrabi
Copy link
Member

¿missing view?

Ah una pregunta español pero con palabras de ingles. 😀

use "ifRoom" and "always" in cases where is possible
Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

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

Thanks again

@TobiGr TobiGr merged commit 5883f6e into TeamNewPipe:dev Aug 12, 2019
This was referenced Aug 13, 2019
@kapodamy kapodamy deleted the buttons-hiding-fix-on-screen-off branch August 22, 2019 22:29
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.

3 participants