Skip to content

align-issue-labels - Disable on beta global PR list#9179

Closed
SunsetTechuila wants to merge 4 commits into
mainfrom
labels-new-pr-list-small
Closed

align-issue-labels - Disable on beta global PR list#9179
SunsetTechuila wants to merge 4 commits into
mainfrom
labels-new-pr-list-small

Conversation

@SunsetTechuila
Copy link
Copy Markdown
Member

@SunsetTechuila SunsetTechuila commented Apr 11, 2026

Too much to account for, changes too frequently

Test URLs

https://github.com/pulls with new experience enabled

Screenshot

@SunsetTechuila SunsetTechuila changed the title align-issue-labels - Fix badges margin on new PR list on mobile align-issue-labels - Fix labels margin on new PR list on mobile Apr 11, 2026
@fregante
Copy link
Copy Markdown
Member

I'd push it 2/4px further to the left to help with optical alignment

@SunsetTechuila
Copy link
Copy Markdown
Member Author

SunsetTechuila commented Apr 12, 2026

We also need to account for this setting:

image

@SunsetTechuila SunsetTechuila marked this pull request as draft April 12, 2026 15:31
@fregante
Copy link
Copy Markdown
Member

We also need to account for this setting:

IMHO that view looks good without Refined GitHub.

As for this PR, I'm seeing the opposite issue on v26.3.21, excessive spacing:

Screenshot 10

@SunsetTechuila SunsetTechuila dismissed stale reviews from Troyans9 April 24, 2026 18:03

spam

@SunsetTechuila SunsetTechuila changed the title align-issue-labels - Fix labels margin on new PR list on mobile align-issue-labels - Disable on beta global PR list Apr 24, 2026
@SunsetTechuila SunsetTechuila marked this pull request as ready for review April 24, 2026 18:15
@fregante
Copy link
Copy Markdown
Member

Do you reckon it's impossible to support the standard view without also affecting the compact view? I'm not sure I want to disable the feature entirely

@SunsetTechuila
Copy link
Copy Markdown
Member Author

blocked by #9260

@SunsetTechuila SunsetTechuila marked this pull request as draft April 24, 2026 19:14
@fregante
Copy link
Copy Markdown
Member

I think this can be done in a single PR, I assume the CSS has to deal with both layouts anyway.

align-issue-labels - Several improvements

@SunsetTechuila
Copy link
Copy Markdown
Member Author

closing in favor of #9260

@SunsetTechuila SunsetTechuila deleted the labels-new-pr-list-small branch April 25, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants