Skip to content

Post title on listing page should always open comments (fixes #3755)#3756

Merged
dessalines merged 1 commit intomainfrom
post-title-click
Jan 15, 2026
Merged

Post title on listing page should always open comments (fixes #3755)#3756
dessalines merged 1 commit intomainfrom
post-title-click

Conversation

@Nutomic
Copy link
Copy Markdown
Member

@Nutomic Nutomic commented Jan 13, 2026

No description provided.


return (
<h1 className="post-name h5 d-inline text-break">
{navigateExternal ? (
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just get of this function then, since its only used here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete the navigateExternal function, its only used in one place, and now that's gone.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its not a function but a variable, and its already removed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops sry, brainfart.


return (
<h1 className="post-name h5 d-inline text-break">
{navigateExternal ? (
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops sry, brainfart.

@dessalines dessalines merged commit 8afd0f6 into main Jan 15, 2026
2 checks passed
@dessalines dessalines deleted the post-title-click branch January 15, 2026 19:48
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.

2 participants