Skip to content

Conversation

@dhanuarf
Copy link
Contributor

For simpler and easier access.

Screenshot_20251026-133423_LibreTube Debug

---------
Also, loading indicators are moved to the top most of the view. And the content's alpha is reduced when it's loading just to create some sort of 'focus' to the loading indicator.

@dhanuarf dhanuarf changed the title feat: add 'change region' button in TrendingFragment feat: add 'change region' button in TrendsFragment Oct 26, 2025
@Cubillas7
Copy link

Hello, does it make sense put a tooltip after long press for this new button? I think that there is no now? Thanksss

@Bnyro
Copy link
Member

Bnyro commented Oct 26, 2025

Hello, does it make sense put a tooltip after long press for this new button? I think that there is no now? Thanksss

Good catch, added with dd241f7

Copy link
Member

@Bnyro Bnyro left a comment

Choose a reason for hiding this comment

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

Thank you @dhanuarf! Just pushed some small changes (i.e. you forgot to re-use showChangeRegionDialog in the HomeFragment, apart from that it looks great!

That had also been on my todo list, but I forgot to look into it :)

@Bnyro Bnyro merged commit 16f7c95 into libre-tube:master Oct 26, 2025
4 checks passed
@dhanuarf dhanuarf deleted the trendregion branch October 26, 2025 11:33
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