Skip to content

Commit 1130acb

Browse files
committed
Fixes commit message headline splitting fallback
- Falls back to splitting commit messages by newline when the autolink splitter token is missing - Ensures stable layout and font sizing before autolinks are resolved and loaded
1 parent 38d9443 commit 1130acb

1 file changed

Lines changed: 13 additions & 2 deletions

File tree

src/webviews/apps/commitDetails/components/gl-details-commit-panel.ts

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -674,7 +674,15 @@ export class GlDetailsCommitPanel extends GlDetailsBase {
674674
if (!commit) return nothing;
675675

676676
const message = this.formattedMessage ?? commit.message;
677-
const index = message.indexOf(messageHeadlineSplitterToken);
677+
// The splitter token is only injected during autolink linkification; the raw commit.message
678+
// (graph commitLite / full fetch) carries a plain newline. Fall back to it so the
679+
// headline/body split — and its font sizing — is stable before autolinks land.
680+
let index = message.indexOf(messageHeadlineSplitterToken);
681+
let bodyOffset = messageHeadlineSplitterToken.length;
682+
if (index === -1) {
683+
index = message.indexOf('\n');
684+
bodyOffset = 1;
685+
}
678686

679687
return html`<div class="message">
680688
<div class="message-block">
@@ -707,7 +715,10 @@ export class GlDetailsCommitPanel extends GlDetailsBase {
707715
<strong
708716
><gl-markdown .markdown=${message.substring(0, index)} density="compact"></gl-markdown
709717
></strong>
710-
<gl-markdown .markdown=${message.substring(index + 3)} density="compact"></gl-markdown>
718+
<gl-markdown
719+
.markdown=${message.substring(index + bodyOffset)}
720+
density="compact"
721+
></gl-markdown>
711722
</div>`,
712723
)}
713724
</div>

0 commit comments

Comments
 (0)