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 icons to trending tab #6239

Conversation

ChunkyProgrammer
Copy link
Member

@ChunkyProgrammer ChunkyProgrammer commented Nov 28, 2024

Add icons to trending tab

Pull Request Type

  • Other - UI change

Description

This PR adds icons to each of the trending tabs

Screenshots

image

Desktop

  • OS: Linux Mint
  • OS Version: 0.22.0
  • FreeTube version: latest nightly

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 28, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) November 28, 2024 14:05
@FreeTubeBot FreeTubeBot merged commit 41d8362 into FreeTubeApp:development Nov 29, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 29, 2024
@ChunkyProgrammer ChunkyProgrammer deleted the add-icons-to-trending-tab branch November 29, 2024 15:28
Copy link
Member

Choose a reason for hiding this comment

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

Not a dealbreaker but i do think that it would be better to change all the icon positions at the same when the space allows for it otherwise users can sometimes see the icons on top of the tab but on the tab to the next of it the icon is still beside next to the tab text

FreeTube_YeDX0ME0ob.mp4

@MarkusGaugg

This comment has been minimized.

Soham456 pushed a commit to Soham456/FreeTube that referenced this pull request Dec 5, 2024
jlvivero pushed a commit to jlvivero/FreeTube that referenced this pull request Dec 7, 2024
SuperAKWA pushed a commit to SuperAKWA/FreeTube that referenced this pull request Jan 24, 2025
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.

7 participants