-
-
Notifications
You must be signed in to change notification settings - Fork 88
Update Readme #322
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
base: main
Are you sure you want to change the base?
Update Readme #322
Conversation
README.md
Outdated
| - Shorts removal | ||
| - Higher-quality thumbnails | ||
| - On-screen clock overlay | ||
| - YouTube logo removal |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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..
|
If you do the following things, this PR will be mergeable as far as I'm concerned.
|
|
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 : after all It is |
0e32aef to
a85e0ed
Compare
a85e0ed to
aff82cb
Compare
|
This is mergeable as long as these 3 are addressed:
|
|
Yeah, 2 & 3 obviously have to be done. Will do. |

https://github.com/jesvijonathan/youtube-webos/tree/Update-Readme
EDIT : [Removed MSG]