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

table enhancement #313

Merged
merged 1 commit into from
Apr 18, 2024
Merged

table enhancement #313

merged 1 commit into from
Apr 18, 2024

Conversation

mawinter69
Copy link
Contributor

@mawinter69 mawinter69 commented Feb 4, 2024

having a separate table for each config file type has the problem that the the column width are different for each type. Also the comment is written in a separate line
This change shows everything in a single table with a tbody for each type. The types are separated by some space and each type has a header with title and column description. Each file is on one line. Styling is moved to a dedicated css file.

remove inline style

Before:
image

After:
image

Testing done

Submitter checklist

having a separate table for each config file type has the problem that
the the column width are different for each type. Also the comment is
written in a separate line
This change shows everything in a single table with a tbody for each type.
The types are separated by some space and each type has a header with
title and column description. Each file is on one line.
Styling is moved to a dedicated css file.

remove inline style
@mawinter69 mawinter69 requested a review from a team as a code owner February 4, 2024 21:02
@monkmachine
Copy link

I like this PR, makes the UI looks so much cleaner

@alecharp
Copy link
Member

I like it as well. Thanks!

@alecharp alecharp added this pull request to the merge queue Apr 18, 2024
Merged via the queue into jenkinsci:master with commit ba80ecb Apr 18, 2024
14 checks passed
@basil
Copy link
Member

basil commented May 13, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants