ui: updates the jobs table styling#70374
Conversation
matthewtodd
left a comment
There was a problem hiding this comment.
Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @maryliag)
This commit updates the style of the table on the Jobs page and adds tooltips to its columns. Resolves cockroachdb#70149 Release note (ui change): Updating job table style to match all other tables on the console and also update column name from `Users` to `User`.
Azhng
left a comment
There was a problem hiding this comment.
🔥 🔥 🔥 !!!
Just curious, I didn't see any CSS change, so was it the original styling issues caused by the tooltips?
Reviewable status:
complete! 1 of 0 LGTMs obtained (and 1 stale) (waiting on @matthewtodd)
maryliag
left a comment
There was a problem hiding this comment.
Different component: now it's using SortedTable from cluster-ui, instead of the one from a more generic component
And then I added the tooltip components on the header of each column
Reviewable status:
complete! 1 of 0 LGTMs obtained (and 1 stale) (waiting on @matthewtodd)
|
bors r+ |
|
Build failed (retrying...): |
|
Build succeeded: |
This commit updates the style of the table on the Jobs page
and adds tooltips to its columns.
Resolves #70149
Before

After

Release note (ui change): Updating job table style to
match all other tables on the console.