Skip to content

Correct labels container position to end of row#245

Merged
m-akinc merged 2 commits intoalejandro5042:masterfrom
kroeschl:labels-position
Apr 15, 2026
Merged

Correct labels container position to end of row#245
m-akinc merged 2 commits intoalejandro5042:masterfrom
kroeschl:labels-position

Conversation

@kroeschl
Copy link
Copy Markdown
Contributor

@kroeschl kroeschl commented Apr 13, 2026

Justification

On the PR dashboard, I noticed that the PR labels are rendered in different locations on different PRs. This looks like an error in the way we're creating the pill container if it's not already present.
Current PR view

Implementation

Add the pills container before the end of the containing element instead of as a sibling after the PR title. Bump the version and update the change notification.

Testing

Testing in my browser, I now see the label pills always rendered after the title. I also sanity-checked the update notification.
Updated PR view

@kroeschl kroeschl marked this pull request as ready for review April 13, 2026 17:42
@m-akinc m-akinc merged commit 76412dc into alejandro5042:master Apr 15, 2026
3 checks passed
@kroeschl kroeschl deleted the labels-position branch April 15, 2026 19:59
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