Skip to content

Conversation

@jesvijonathan
Copy link

@jesvijonathan jesvijonathan commented Aug 4, 2025

README.md Outdated
- Shorts removal
- Higher-quality thumbnails
- On-screen clock overlay
- YouTube logo removal
Copy link
Member

Choose a reason for hiding this comment

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

This should have consistent capitalization and only list features that have been merged.

Copy link
Author

Choose a reason for hiding this comment

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

ye, lets merge the readme after all the mentioned have been merged..

@throwaway96
Copy link
Member

If you do the following things, this PR will be mergeable as far as I'm concerned.

  • Fix capitalization of "webOS" in 1_sm.jpg and "WebOS Homebrew Channel" link
  • Change the heading to "YouTube Ad-Free" and the part below the badges to "YouTube app for webOS TV with ad blocking and other enhancements"
  • Remove references to features that have not been merged (like "Audio-Only Mode" and "Magic Remote Scroll-Based Video Seeking")

@jesvijonathan
Copy link
Author

jesvijonathan commented Aug 5, 2025

This, I can't do,"Fix capitalization of "webOS" in 1_sm.jpg ". it looks ugly af. I'm pretty good with UI and I will tell you this it gives me PTSD seeing it that way. Give me something else, maybe "Youtube Ad-Free Menu", looks short and sweet.

I see no issue with other remarks, although you are adding more work later on by removing the features list... I would say its better to wait and release this as a v5 milestone with all the other new features, this way it would be planed good I would say... unless you want more unnecessary commits later on, but okay.. not my repo.. I have removed it

one thing we could do is change :
make it __ytaf_title__ + "Menu" and take the value from appinfo.json like with version in #325

after all It is
image
in appinfo.json. Youtube AdFree Menu sounds good

@throwaway96 throwaway96 added this to the v0.5.0 milestone Aug 9, 2025
@fire332
Copy link
Collaborator

fire332 commented Oct 10, 2025

This is mergeable as long as these 3 are addressed:

  1. From @throwaway96's earlier comment:
  • Remove references to features that have not been merged (like "Magic Remote Scroll-Based Video Seeking")
  1. The screenshot needs to be updated to reflect Updated extended menu colour match & Added 'youtube-webos' version #325.
  2. Rebase this branch on main.

@jesvijonathan
Copy link
Author

Yeah, 2 & 3 obviously have to be done. Will do.
About 1, lets wait. Just one last feature & that too I use this alot btw, People will love it, trust me.

@fire332 fire332 removed this from the v0.5.0 milestone Nov 10, 2025
@fire332 fire332 added the documentation Improvements or additions to documentation label Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants