Skip to content

Fix Disable Channel Links in Channel Sidenav Menu - #9395

Open
ozrendev wants to merge 2 commits into
FreeTubeApp:developmentfrom
ozrendev:disable-links-channel-list
Open

Fix Disable Channel Links in Channel Sidenav Menu#9395
ozrendev wants to merge 2 commits into
FreeTubeApp:developmentfrom
ozrendev:disable-links-channel-list

Conversation

@ozrendev

Copy link
Copy Markdown
Contributor

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

relates to #8502

Description

These links were missed in the original PR, so this disables them as originally intended

Testing

Go to Settings -> Parental Control -> Enabled "Disable Channel Link on Videos"
Go to Channels sidenav item, see links are disabled for Channel thumbnail and name

@FreeTubeBot
FreeTubeBot enabled auto-merge (squash) July 10, 2026 06:01
@github-actions github-actions Bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jul 10, 2026
@PikachuEXE

Copy link
Copy Markdown
Member

Should we also disable the icons on side bar? (PR disable channel links in Channel List
image

auto-merge was automatically disabled July 11, 2026 01:34

Head branch was pushed to by a user without write access

@FreeTubeBot
FreeTubeBot enabled auto-merge (squash) July 11, 2026 01:34
@ozrendev

Copy link
Copy Markdown
Contributor Author

Should we also disable the icons on side bar? (PR disable channel links in Channel List

Yes, good catch! Added that now

@efb4f5ff-1298-471a-8973-3d47447115dc

Copy link
Copy Markdown
Member

Umm i dont know if thats really necessary as there is already a setting to hide them. Making them non functional feels not logical to me but ofcourse my feeling could be wrong here

@PikachuEXE

Copy link
Copy Markdown
Member

Well my reason would be the existing setting is under "distraction free" not parental control
And we should provide less surprise for users ("what? I need 2 settings to do what I want?
Now whether those sidebar icons should be displayed at all when the new setting is enabled is something else (I have no idea

@efb4f5ff-1298-471a-8973-3d47447115dc

Copy link
Copy Markdown
Member

I guess you're right. I didnt put myself in users shoes enough

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should the hover and click animation be disabled too here?

VirtualBoxVM_X4RSr56cFA.mp4

@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 14 days.

@PikachuEXE

Copy link
Copy Markdown
Member

should the hover and click animation be disabled too here?

ya

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants