diff --git a/src/features/pr-review/PRDescription.svelte b/src/features/pr-review/PRDescription.svelte index 718c8d6..77831ba 100644 --- a/src/features/pr-review/PRDescription.svelte +++ b/src/features/pr-review/PRDescription.svelte @@ -55,16 +55,12 @@ const cleanText = $derived.by(() => stripMarkdown(displayText)); const renderedMarkdown = $derived.by(() => { - // Always render markdown when expanded, or when there's no toggle needed - if ((expanded && shouldShowToggle) || !shouldShowToggle) { - try { - return marked.parse(displayText); - } catch (error) { - console.error('Error rendering markdown:', error); - return displayText; - } + try { + return marked.parse(displayText); + } catch (error) { + console.error('Error rendering markdown:', error); + return displayText; } - return null; }); // Check if the content has markdown formatting @@ -84,13 +80,11 @@
- {#if hasMarkdown && (renderedMarkdown || (!shouldShowToggle && !expanded))} - + {#if hasMarkdown}
{@html renderedMarkdown}
{:else} -
{cleanText}
diff --git a/src/features/pr-review/components/ReviewSubmissionSection.svelte b/src/features/pr-review/components/ReviewSubmissionSection.svelte index af7f6c0..e65ba1e 100644 --- a/src/features/pr-review/components/ReviewSubmissionSection.svelte +++ b/src/features/pr-review/components/ReviewSubmissionSection.svelte @@ -52,11 +52,11 @@
-
+
+ +
+ {/if} +
{/if} @@ -82,4 +128,34 @@ .breadcrumb-current { color: #d1d5db; } + + .pr-nav-btn { + display: inline-flex; + align-items: center; + gap: 0.25rem; + padding: 0.25rem 0.625rem; + border-radius: 0.375rem; + border: 1px solid #4b5563; + color: #d1d5db; + font-size: 0.8125rem; + font-weight: 500; + transition: all 0.2s ease; + background-color: rgba(55, 65, 81, 0.5); + } + + .pr-nav-btn:hover:not(:disabled) { + background-color: rgba(59, 130, 246, 0.15); + border-color: #60a5fa; + color: #60a5fa; + } + + .pr-nav-btn:disabled { + opacity: 0.3; + cursor: not-allowed; + } + + .pr-nav-btn:focus { + outline: none; + box-shadow: 0 0 0 2px rgba(59, 130, 246, 0.5); + }