-
-
Notifications
You must be signed in to change notification settings - Fork 296
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] Remove the quote-inline
class from statuses
#2867
Comments
Could you give an example? It's not totally clear from the issue what's being removed, what's not being removed, and what should be kept. |
I think there's an example in GeopJr/Tuba#926 (comment), if you expand the "Details" thingy. |
Oh OK, we could probably just add that to our sanitizer then. |
Yep, I'll include it here too: GTS:
Mastodon:
diff: - experience<span class=""><br><br>RE: <a href=
+ experience<span class="quote-inline"><br><br>RE: <a href= edit: I want So in the case of Tuba, since we do display quoted posts when provided, we remove any elements that have the |
Got it, thanks for the extra detail 👍 |
Is your feature request related to a problem ?
When *oma, *key(?) and treehouse.systems mastodon do a quote post, they wrap
RE: <link>
in a span with the classquote-inline
, so clients hide that line if they support quote posts. GTS seems to be keeping them while removing the quote, ending up in quotes without a link to the originalDescribe the solution you'd like.
quote-inline
gets removed until quotes get implemented (if at all)Describe alternatives you've considered.
From my client's side, I'll check if there's a quote property available before removing the
RE: ...
sentenceAdditional context.
GeopJr/Tuba#926
The text was updated successfully, but these errors were encountered: