Skip to content

Meta: Avoid unnecessary JSX fragments - #9537

Merged
fregante merged 13 commits into
mainfrom
no-fragments
May 20, 2026
Merged

fregante merged 13 commits into
mainfrom
no-fragments

Conversation

@fregante fregante changed the title Meta: Avoid unnecessary JSX fragments #9522 Meta: Avoid unnecessary JSX fragments May 20, 2026

@fregante fregante left a comment •

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.

rgh-feature-descriptions received the most changes, as well as some functional changes.

The rest are rather straightforward search-replace type changes.

'1 open issue',
'$$ open issues',
'No open issues',
'Related issues',

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.

Regression in the last cleanup PR

return `is:open (${searchTerms})`;
const searchTerms = [id, ...oldNames].map(name => `"${name}"`);
const joinedTerms = searchTerms.length > 1 ? `(${searchTerms.join(' OR ')})` : searchTerms[0];
return `is:open ${joinedTerms}`;

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.

I did not like the search is:open ("single-name")

Comment on lines +59 to +69
if (wasFeatureRemoved) {
links.push(
// This links to the full commit history, which will start with the commit that removed the file
<a
data-turbo-frame="repo-content-turbo-frame"
href={`https://github.com/refined-github/refined-github/commits/main/source/features/${id}.tsx`}
>
Commit history
</a>,
);
}

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.

Before

Image

After

Image

@fregante fregante added the meta Related to Refined GitHub itself label May 20, 2026
@fregante
fregante merged commit 37ffb8e into main May 20, 2026
10 checks passed
@SunsetTechuila
SunsetTechuila deleted the no-fragments branch May 20, 2026 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Related to Refined GitHub itself

Development

Successfully merging this pull request may close these issues.

1 participant