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

Correct hashtag warning #22827

Merged
merged 1 commit into from
Jan 9, 2023
Merged

Correct hashtag warning #22827

merged 1 commit into from
Jan 9, 2023

Conversation

n0toose
Copy link
Contributor

@n0toose n0toose commented Dec 29, 2022

Posts with any visibility setting that is not 'Public' are prevented from being listed under any hashtag.


P.S. I did not update the string in other locales, e.g. en-GB, under the assumption that this would be done by the translators and because of similar pull requests by reputable contributors not containing such changes. Hope that's OK.

Posts with any visibility setting that is not 'Public' are prevented
from being listed under any hashtag.
@ClearlyClaire
Copy link
Contributor

Posts with any visibility setting that is not 'Public' are prevented from being listed under any hashtag.

I personally think the current wording is fine, it recalls the selected visibility (unlisted), and says that only posts with public visibility are listed in the hashtag timelines. I feel like with your change, the first sentence is just paraphrasing the second one, while the current wording recalls the selected visibility.

P.S. I did not update the string in other locales, e.g. en-GB, under the assumption that this would be done by the translators and because of similar pull requests by reputable contributors not containing such changes. Hope that's OK.

Yes, that's the way to go! Translations are managed via Crowdin, so changing translations in PRs are to be avoided as this is likely to cause merge conflicts when merging the translation updates from Crowdin.

@ClearlyClaire
Copy link
Contributor

Posts with any visibility setting that is not 'Public' are prevented from being listed under any hashtag.

I personally think the current wording is fine, it recalls the selected visibility (unlisted), and says that only posts with public visibility are listed in the hashtag timelines. I feel like with your change, the first sentence is just paraphrasing the second one, while the current wording recalls the selected visibility.

Oh, nevermind… I somehow missed that it's shown for other selected visibilities as well 😅

I think it could recall the selected visibility, but that's probably more trouble than it's worth. Your proposal is fine!

@n0toose
Copy link
Contributor Author

n0toose commented Jan 5, 2023

I think it could recall the selected visibility, but that's probably more trouble than it's worth. Your proposal is fine!

Would've done the same, but there's a very good reason why I didn't choose to implement this: I don't know Ruby. :D

@Gargron Gargron merged commit 105e1f0 into mastodon:main Jan 9, 2023
nametoolong pushed a commit to nametoolong/nuage that referenced this pull request Jan 12, 2023
Posts with any visibility setting that is not 'Public' are prevented
from being listed under any hashtag.
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.

3 participants