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

[Feature Request] Leave words together on web previews #39

Open
sabogalc opened this issue May 11, 2021 · 1 comment
Open

[Feature Request] Leave words together on web previews #39

sabogalc opened this issue May 11, 2021 · 1 comment

Comments

@sabogalc
Copy link

For example, in the photo below, the word "did" should be on the second line instead of the first letter being on the first line and the other two being on the second line.
image
or it could also be centered/evenly split like it is on iOS
PNG image

Another thing to note is that the link is a mobile.twitter.com link, but the frontend points it to twitter.com. You can see the mobile.twitter.com link in the conversation preview and the twitter.com link in the bottom left (issue #34)
Issue 34

While for this instance, it doesn't make a difference as both web pages show the same thing (much like youtu.be vs. youtube.com)
image
image

It does make a difference for a page like Wikipedia.
image
image

So, in cases where a mobile link is converted to a desktop link, does issue #34 become a bug that is "not a bug, but a feature"? (However, I feel like the GitHub comment example from issue #34 should still be fixed).

@sabogalc
Copy link
Author

sabogalc commented May 11, 2021

SMServer correctly points to the mobile.twitter.com link (though the web preview just says twitter.com), but it is missing the actual tweet and the Twitter icon.
SMServer
WebMessage spaces out the words in the tweet correctly (the original point of this ticket), but it is missing the Twitter icon, and it also goes to twitter.com instead of mobile.twitter.com
image

Edit - The WebMessage window can be resized, but even when it is resized, it never splits up words.
image

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

No branches or pull requests

1 participant