Skip to content

Conversation

@shuuji3
Copy link
Member

@shuuji3 shuuji3 commented Jan 8, 2026

Mastodon prepends the "RE: https://..." string in quote post for the compatibility purpose. This PR adjust the existing parser to allow p.quote-inline element and hides it.

Before

Screenshot of quote post, showing a text 'RE: https://...' by mastodon's compat purpose

After

Screenshot of quote post without RE text

@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for elk-docs canceled.

Name Link
🔨 Latest commit 443a0da
🔍 Latest deploy log https://app.netlify.com/projects/elk-docs/deploys/695f42e795c44d0008d42446

@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for elk-zone ready!

Name Link
🔨 Latest commit 443a0da
🔍 Latest deploy log https://app.netlify.com/projects/elk-zone/deploys/695f42e775ad6c000812cdfa
😎 Deploy Preview https://deploy-preview-3504--elk-zone.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@shuuji3
Copy link
Member Author

shuuji3 commented Jan 8, 2026

confirmed the fix!

@shuuji3 shuuji3 added this pull request to the merge queue Jan 8, 2026
Merged via the queue into main with commit 6620b17 Jan 8, 2026
21 checks passed
@shuuji3 shuuji3 deleted the shuuji3/feat/hide-compat-quote-url branch January 8, 2026 05:47
@cheeaun
Copy link
Contributor

cheeaun commented Jan 8, 2026

I think it should only be hidden if there's a native quote.

@shuuji3 shuuji3 linked an issue Jan 9, 2026 that may be closed by this pull request
@shuuji3
Copy link
Member Author

shuuji3 commented Jan 9, 2026

Good point.😅 My assumption was class="quote-inline" is only added to <p>RE: ...</p> element, just based on my observation that only Matsodon v4.5 add class="quote-inline" but Mastodon v4.4 doesn't (not sure if there's documentation). But I think checking that can prevent hiding it unintentionally.

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.

Hide "RE: https://..." string in quote post

3 participants