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

Display “Show newer” and “Show older” instead of “Show more” in public pages #15052

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

ariasuni
Copy link
Contributor

It allows for better translations (it was limiting me in French) and it makes clearer what the “order” of the toots is.

On a profile page:
image
image

On a thread page:
image
image

@@ -17,7 +17,7 @@
- if status.reply? && include_threads
- if @next_ancestor
.entry{ class: entry_classes }
= link_to_more ActivityPub::TagManager.instance.url_for(@next_ancestor)
= link_to_older ActivityPub::TagManager.instance.url_for(@next_ancestor)
Copy link
Member

Choose a reason for hiding this comment

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

Hm, in case of threads, I am not sure if older/newer makes more sense than just "more". It's true that descendants are "newer" and ancestors are "older", but it may still be confusing. I'm not sure. Would love to hear what others think.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's good either way, though showing just “more” would require yet another translatable string.

@Gargron Gargron merged commit 68d4b2b into mastodon:master Nov 4, 2020
umonaca pushed a commit to umonaca/mastodon that referenced this pull request Nov 8, 2020
@ariasuni ariasuni deleted the disambiguate-show-more branch January 31, 2021 21:01
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