Render embedded quoted posts as blockquotes with handle permalink
Fixes #1. Replaces the plain div wrapper and separate "View on Bluesky" link with a <blockquote> element, and makes the @handle the permalink to the quoted post. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit was merged in pull request #3.
This commit is contained in:
@@ -376,17 +376,14 @@ final class BlueskyThreadsExtension extends Minz_Extension {
|
||||
}
|
||||
|
||||
return <<<HTML
|
||||
<div style="border:1px solid #0085ff;border-radius:12px;padding:10px 12px;margin-top:8px;">
|
||||
<blockquote style="border:1px solid #0085ff;border-radius:12px;padding:10px 12px;margin:8px 0 0 0;">
|
||||
<div style="font-size:0.85em;margin-bottom:6px;">
|
||||
<strong>{$displayName}</strong>
|
||||
<span style="color:#536471;margin-left:4px;">@{$this->e($handle)}</span>
|
||||
<a href="{$this->e($postUrl)}" style="color:#536471;margin-left:4px;text-decoration:none;">@{$this->e($handle)}</a>
|
||||
</div>
|
||||
<div style="white-space:pre-wrap;word-break:break-word;">{$text}</div>
|
||||
{$embedsHtml}
|
||||
<div style="margin-top:6px;font-size:0.75em;">
|
||||
<a href="{$this->e($postUrl)}" style="color:#536471;">View on Bluesky ↗</a>
|
||||
</div>
|
||||
</div>
|
||||
</blockquote>
|
||||
HTML;
|
||||
}
|
||||
|
||||
|
||||
@@ -164,14 +164,15 @@ class BlueskyThreadsTest extends TestCase {
|
||||
'Quoted post text must be rendered as an embed');
|
||||
}
|
||||
|
||||
/** Quoted posts include a "View on Bluesky" link; thread posts do not. */
|
||||
/** Quoted posts use a blockquote with a handle permalink; thread posts have no permalink. */
|
||||
public function testViewOnBskyLinksArePresent(): void {
|
||||
$data = json_decode(file_get_contents(__DIR__ . '/fixtures/thread_3mhtk7awhrp26.json'), true);
|
||||
$html = $this->call('renderThread', $data['thread'], true);
|
||||
|
||||
$this->assertStringNotContainsString('3mhtk7awhrp26', $html, 'Thread post rkey must not appear as a permalink');
|
||||
$this->assertStringContainsString('3mhtjo3rtkn26', $html, 'Quoted post rkey must appear in its View on Bluesky link');
|
||||
$this->assertStringContainsString('View on Bluesky', $html);
|
||||
$this->assertStringContainsString('3mhtjo3rtkn26', $html, 'Quoted post rkey must appear in the handle permalink');
|
||||
$this->assertStringContainsString('<blockquote', $html, 'Quoted post must be wrapped in a blockquote');
|
||||
$this->assertStringNotContainsString('View on Bluesky', $html);
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user