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

feat: option to collapse long notes #8561

Merged

Conversation

yitsushi
Copy link
Contributor

@yitsushi yitsushi commented Apr 28, 2022

What

Add an option to re-collapse long messages.

Why

Opening long messages on the timeline can make it hard to scroll back and forth, that mostly results in a "meh, I'll just refresh the page"

Additional info (optional)

Please give me a string value for "Show less" in Japanese, Got a tip to use "閉じる", if it's not the right one, let me know. As I understand, the new keyword will get into the flow if it has a Japanese value, the rest come from Crowdin (that's why I did not commit any other languages/translations).

Closes #8559

@yitsushi
Copy link
Contributor Author

added en-US manually while testing, did not commit that file as it will be overwritten by Crowdin.

image

It has no effect on other variations (short message, has cw)

image

@yitsushi
Copy link
Contributor Author

yitsushi commented Apr 29, 2022

It was a bit hard to find something that makes it well visible on both light and dark themes.
image
image

@tamaina tamaina requested a review from syuilo May 5, 2022 13:42
@syuilo syuilo merged commit e675ffc into misskey-dev:develop Jun 4, 2022
@syuilo
Copy link
Member

syuilo commented Jun 4, 2022

👍

syuilo added a commit that referenced this pull request Jun 4, 2022
@Johann150
Copy link
Contributor

Why was it reverted?

@syuilo
Copy link
Member

syuilo commented Jun 4, 2022

There was a problem with URL previews and images being cropped.
I am considering how to handle this issue.

@Johann150
Copy link
Contributor

I tried with images, with link previews, and with both but I did not notice any problems. 😕

Johann150 added a commit that referenced this pull request Jul 13, 2022
This reverts commit 71150f2 and reinstates
the option to collapse long notes again after they were expanded.
@Johann150 Johann150 mentioned this pull request Jul 13, 2022
syuilo added a commit that referenced this pull request Aug 1, 2022
* Revert "Revert option to collapse long notes (#8561)"

This reverts commit 71150f2 and reinstates
the option to collapse long notes again after they were expanded.

* fix(client): wrongly collapsed posts

* fix: don't use ref

* tweak style

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
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.

option to collapse long notes
3 participants