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

ui: add horizontal scroll txn insight details #91283

Merged
merged 1 commit into from
Nov 8, 2022

Conversation

maryliag
Copy link
Contributor

@maryliag maryliag commented Nov 4, 2022

Previously, the table of "waiten on" inside the Transaction Insights details didn't have a horizontal scroll, not letting the user to see the full page on CC console. This commits adds the proper scroll to it.

Fixes #91199

Before
https://www.loom.com/share/80437381cd4546cfad9692c7718de38c

After
https://www.loom.com/share/7363d53bec1d49a0be1b90ab0c9069f2

Release note (bug fix): Add horizontal scroll on "waited on" table on Transaction Insight details page.

@maryliag maryliag requested a review from a team November 4, 2022 15:09
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Previously, the table of "waiten on" inside the Transaction
Insights details didn't have a horizontal scroll, not
letting the user to see the full page on CC console.
This commits adds the proper scroll to it.

Fixes cockroachdb#91199

Release note (bug fix): Add horizontal scroll on "waited on"
table on Transaction Insight details page.
@maryliag maryliag force-pushed the scroll-txn-insights branch from 2ae8250 to af4d9da Compare November 4, 2022 19:39
Copy link
Contributor

@j82w j82w left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @maryliag)

@maryliag
Copy link
Contributor Author

maryliag commented Nov 7, 2022

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 7, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 7, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 7, 2022

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 7, 2022

Build failed (retrying...):

@maryliag
Copy link
Contributor Author

maryliag commented Nov 8, 2022

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 8, 2022

Already running a review

@maryliag
Copy link
Contributor Author

maryliag commented Nov 8, 2022

bors r-

@craig
Copy link
Contributor

craig bot commented Nov 8, 2022

Canceled.

@maryliag
Copy link
Contributor Author

maryliag commented Nov 8, 2022

bors r+

@craig craig bot merged commit 666fdee into cockroachdb:master Nov 8, 2022
@craig
Copy link
Contributor

craig bot commented Nov 8, 2022

Build succeeded:

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.

ui: the transaction insight details "waited on" table can be cut off
3 participants