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

Move kodi button to better position #471

Closed
2 tasks done
mueller-ma opened this issue Mar 3, 2017 · 11 comments
Closed
2 tasks done

Move kodi button to better position #471

mueller-ma opened this issue Mar 3, 2017 · 11 comments

Comments

@mueller-ma
Copy link
Contributor

  • I carefully read the contribution guidelines and agree to them.
  • I checked if the issue/feature exists in the latest version.

Hi,

I really like the "play on kodi" button, however it is in the dropdown menu and not accessable as quick as it could be. I have appended an image with the kodi button on a better position. In this case it might be better to diable the feature by default, because people could get distraced by a second play button without purpose for them.

newpipe-kodi

@mueller-ma
Copy link
Contributor Author

@mauriciocolli
Copy link
Contributor

I was thinking in doing something like that, I guess the share is not as important as an option of the user, so it can be in the dropdown if it don't have enough space.

You can open a pull request, but keep in mind that when you enable/disable the option in the settings, the option are not appearing unless the application is restarted (or exiting the video fragment, and enter again), so it requires another change as I forgot to include this button/option as an update of the UI.

And I'll open a PR fixing this soon.

@theScrabi
Copy link
Member

@mauriciocolli as long as you keep leting the user decide weather he wants to see the share or the kodi button in the action bar I'm fine with that :)

@mauriciocolli
Copy link
Contributor

@theScrabi I guess, who enable the Kodi will want easier access (use often/constantly) to that feature than the share and it's not like the share will be gone, so I think an option just for that would be overkill.

What do you think?

@mueller-ma
Copy link
Contributor Author

Can we use this: Share, Kodi, Download and change Download to "ifRoom" and Share to "always"

@theScrabi
Copy link
Member

All right, I'm good with that :)

@mueller-ma
Copy link
Contributor Author

I make a PR

@mauriciocolli
Copy link
Contributor

I think that Download should always be visible in the actionbar, as it is one of our "primary" features.

And just changing the order of the Play on Kodi and Share is enough...

@theScrabi
Copy link
Member

@mauriciocolli yea you are right. Of course the download must stay visible, just that slot aside of it can vary.

@mueller-ma
Copy link
Contributor Author

@mauriciocolli @theScrabi I think when someone manually enables the kodi button than it is more important than the download

@mauriciocolli
Copy link
Contributor

@mueller-ma that's what I said in the previous post, except that it was to "replace" the share.

But the download button HAVE to be the first button and always be visible, the share that is interchangeable. And I think in all phone made in the last years (the screen size average) will be showing 2 items, so there's no problem leaving the Download button as first.

mauriciocolli pushed a commit to mauriciocolli/NewPipe that referenced this issue May 14, 2017
mauriciocolli pushed a commit to mauriciocolli/NewPipe that referenced this issue May 14, 2017
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

No branches or pull requests

3 participants