Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-74783] Fix job name line break behavior in Firefox #492

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

daniel-beck
Copy link
Member

@daniel-beck daniel-beck commented Oct 29, 2024

#476 evidently was not tested in Firefox, which has awkward line break behavior with an inline-flex. https://ci.jenkins.io/job/Core/job/jenkins/view/change-requests/ looks like this:

Screenshot 2024-10-28 at 18 03 27

This changes the column to be more consistent with https://github.com/jenkinsci/jenkins/blob/e1f99468fd81d6aa52dc434fefc6e0420e1cb449/core/src/main/resources/hudson/views/JobColumn/column.jelly#L29.

Tested only with browser dev tools in Firefox so far.

Lazy branch, please delete on merge if GH doesn't do it.

@daniel-beck daniel-beck requested a review from a team as a code owner October 29, 2024 08:56
@mawinter69
Copy link

@daniel-beck daniel-beck changed the title Fix job name line break behavior in Firefox [JENKINS-74783] Fix job name line break behavior in Firefox Oct 29, 2024
@daniel-beck
Copy link
Member Author

daniel-beck commented Oct 29, 2024

@mawinter69 Thanks, updated PR title and linked from there to here.

@rsandell rsandell added the bug label Oct 29, 2024
Copy link
Member

@timja timja left a comment

Choose a reason for hiding this comment

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

LGTM

@jglick jglick merged commit 59ce1bb into master Oct 29, 2024
18 checks passed
@jglick jglick deleted the fixup-476 branch October 29, 2024 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants