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

Better description for hiding watched vids #6382

Closed
wants to merge 1 commit into from

Conversation

lamteteeow
Copy link

@lamteteeow lamteteeow commented Dec 14, 2024

Title

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

closes #6389

Description

Hide Videos on Watch confused me and very likely others too so I made some minor changes to the text.

The reason why it should be Videos marked as WATCHED and not simply watched Videos is because of future features. If there is an option for user to mark a video manually as WATCHED (probably watched somewhere else) then this button should also apply.

Screenshots

Testing

Desktop

  • OS: Windows
  • OS Version: 23H2
  • FreeTube version: v0.22.1 Beta

Additional context

…videos

`Hide Videos on Watch` confused me and very likely others too so I made some minor changes.
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 14, 2024 22:01
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 14, 2024
@lamteteeow lamteteeow changed the title fix(subscription-settings.vue) Better description for hiding watched vids Better description for hiding watched vids Dec 16, 2024
@ChunkyProgrammer
Copy link
Member

I'd recommend keeping the label the same and only updating the value here (otherwise you'll have to update the label for all languages):

Hide Videos on Watch: Hide Videos on Watch

@kommunarr
Copy link
Collaborator

I also do not think we need anything in all-caps or otherwise. I would contend that the existing label is good at communicating the constancy of the setting, and also that the use case of manually marking videos as watched (as is currently possible) is implied enough such as to not necessitate special wording.

@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 Dec 16, 2024
@efb4f5ff-1298-471a-8973-3d47447115dc

I agree with @kommunarr here i dont see a reason to change this.

confused me and very likely others too

If you're confused about it, that's fine. However, trying to strengthen your argument by making invalid claims about what others are thinking doesn't hold up

@lamteteeow
Copy link
Author

lamteteeow commented Dec 16, 2024

If you're confused about it, that's fine. However, trying to strengthen your argument by making invalid claims about what others are thinking doesn't hold up.

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

So far searching through the Issue section, the term Hide Videos on Watch is mostly used by developers/maintainers/members. Others have created their issues using different ways (to me more intuitive) to call it, which exist in #1362 #5261 #4863 #5189 #4497 #5500, etc. Most of them are Hide watched videos or related.

I agree with @kommunarr, Hide watched videos seems more concise and accurately reflects the term users are using.

I love the app, played around with it and only then figured out what that button does. If this seems more like a "me" problem then feel free to close the PR. Otherwise let me know if I should continue as @ChunkyProgrammer recommended.

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

I got this issue while sitting together with a German friend, hence the statement. Changed language to German, same result for both.

Calling me out for something I carefully wrote very likely seems pedantic to me.

Over the past years i have seen allot of people making claims about others when they want something implemented. So i made an assumption here and that was wrong of me but i do stand firm on the argument that you should have made it clear why you said that in the first place. You cannot just say something without proper argumentation

So far searching through the Issue section, the term Hide Videos on Watch is mostly used by developers/maintainers/members. Others have created their issues using different ways (to me more intuitive) to call it, which exist in #1362 #5261 #4863 #5189 #4497 #5500, etc. Most of them are Hide watched videos or related.

This boils down to a subjective change and still dont see a reason to change this

@lamteteeow
Copy link
Author

I was thinking it should be better for everyone. In that case, I will close the PR. Thanks for answering.

@lamteteeow lamteteeow closed this Dec 18, 2024
auto-merge was automatically disabled December 18, 2024 07:48

Pull request was closed

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.

[Feature Request]: Better description for hiding watched vids
4 participants