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

Rename "All" to "All Known Communities" #29

Closed
ShadowJonathan opened this issue Jun 12, 2023 · 12 comments
Closed

Rename "All" to "All Known Communities" #29

ShadowJonathan opened this issue Jun 12, 2023 · 12 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@ShadowJonathan
Copy link
Contributor

After seeing this post, I think it'd be good to clarify that "All" does not mean "All Servers", but "All Known Communities", which is limited by the local server, and all the communities that any local user has subscribed to.

@ShadowJonathan ShadowJonathan added the bug Something isn't working label Jun 12, 2023
@EricBAndrews EricBAndrews added the good first issue Good for newcomers label Jun 27, 2023
@jiannazzone
Copy link

jiannazzone commented Jun 28, 2023

I'll take this one as my first task if it's not already in progress. I have some solo Swift apps, but no experience working as part of a team like this.

@EricBAndrews EricBAndrews added enhancement New feature or request and removed bug Something isn't working labels Jun 30, 2023
@ShadowJonathan ShadowJonathan transferred this issue from mlemgroup/mormaer-mlem Jul 3, 2023
@ghost
Copy link

ghost commented Jul 11, 2023

image
@ShadowJonathan like this? just an update to the description?

@ShadowJonathan
Copy link
Contributor Author

No, change "All Communities" to "Known Communities", and then it'll work

@ghost
Copy link

ghost commented Jul 11, 2023

image
image
@ShadowJonathan

@Sjmarf
Copy link
Contributor

Sjmarf commented Jul 12, 2023

"All Known Communities" would work better imo.

@ghost
Copy link

ghost commented Jul 12, 2023

Yea, I would totally be fine with a description change personally versus changing the headline.

But I just wanna clarify the feed name versus the communities name, as they're intrinsically linked thru an enum. I'd just have to work around it depending on the names.

@ShadowJonathan what would the end names be.

Communities .all =
Feed .all =

@ShadowJonathan
Copy link
Contributor Author

ShadowJonathan commented Jul 12, 2023

Known, the problem with "all" is that it reinforces the idea that the local server (or app) is enumerating all communities "on Lemmy", while that isn't the case.

Description is fine, but imo All -> Known.

Possibly "All Known" for the feed header would work, yeah.

@Sjmarf
Copy link
Contributor

Sjmarf commented Jul 12, 2023

Known, the problem with "all" is that it reinforces the idea that the local server (or app) is enumerating all communities "on Lemmy", while that isn't the case.

Description is fine, but imo All -> Known.

Possibly "All Known" for the feed header would work, yeah.

The reason why I think we should retain the word 'all' is for consistency with Lemmy-ui. "All known communities" is clear enough imo.

@ShadowJonathan
Copy link
Contributor Author

Ah, in that case i think "All Known" across the board could be better. I was a bit afraid that "All" in this instance would simply shortcut someone into thinking that it is "All", not "All Known"

Sorry for the churn, "All Known" it is.

@ghost
Copy link

ghost commented Jul 12, 2023

image
image
TA-DA!

@ghost
Copy link

ghost commented Jul 12, 2023

May I be assigned this task, this is my first contribution.

@ShadowJonathan ShadowJonathan assigned ghost Jul 12, 2023
@EricBAndrews
Copy link
Member

Fixed in #274

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Archived in project
Development

No branches or pull requests

4 participants