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

Add back button in the integrated player #901

Merged
merged 5 commits into from
Nov 25, 2022

Conversation

susonthapa
Copy link
Contributor

Changes
I have added the back button functionality to the integrated player. I have used a Toolbar to achieve this. I was thinking of just adding a simple button but at this point, I feel like I'm reimplementing the Toolbar so I just replaced the TextView with the Toolbar.

Issues
Closes #896

Before
SCR-20221123-e4g

After
SCR-20221123-eaj

@jellyfin-bot jellyfin-bot added this to the v2.5.0 milestone Nov 23, 2022
Copy link
Member

@Maxr1998 Maxr1998 left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!
Did some small tweaks myself, and thus I can merge this now.

@Maxr1998 Maxr1998 enabled auto-merge (squash) November 25, 2022 22:12
@Maxr1998 Maxr1998 merged commit a10c54a into jellyfin:master Nov 25, 2022
@Maxr1998 Maxr1998 added the enhancement New feature or request label May 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suggestions for the integrated player
3 participants