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: fix columns selector being cut #104670

Merged
merged 1 commit into from
Jun 12, 2023
Merged

Conversation

maryliag
Copy link
Contributor

@maryliag maryliag commented Jun 9, 2023

Previously, when the number of rows on the Jobs table was smaller than 3, the height of the area was not enough to show the columns selector, making it get cut and not displayed the "Apply" button unless you scrolled.
This commit adds a min-height for that area, making sure the column selector will be displayed completely.

Fixes #104619

Before
Screenshot 2023-06-09 at 12 09 57 PM

After
Screenshot 2023-06-09 at 12 17 14 PM

Release note (bug fix): The column selector on the Jobs page is no longer getting cut.

Previously, when the number of rows on the Jobs table was smaller
than 3, the height of the area was not enough to show the
columns selector, making it get cut and not displayed the "Apply"
button unless you scrolled.
This commit adds a min-height for that area, making sure the
column selector will be displayed completely.

Fixes cockroachdb#104619

Release note (bug fix): The column selector on the Jobs page
is no longer getting cut.
@maryliag maryliag added backport-22.2.x backport-23.1.x Flags PRs that need to be backported to 23.1 labels Jun 9, 2023
@maryliag maryliag requested review from a team June 9, 2023 16:22
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@gtr gtr left a comment

Choose a reason for hiding this comment

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

Nice! :lgtm_strong:

Reviewed 1 of 1 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 Jun 9, 2023

TFTR!
bors r+

@craig
Copy link
Contributor

craig bot commented Jun 9, 2023

This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried.

Additional information:

{"message":"Changes must be made through a pull request.","documentation_url":"https://docs.github.com/articles/about-protected-branches"}

@maryliag
Copy link
Contributor Author

maryliag commented Jun 9, 2023

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 9, 2023

Build failed:

@maryliag
Copy link
Contributor Author

maryliag commented Jun 9, 2023

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 9, 2023

Build failed:

@maryliag
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 12, 2023

This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried.

Additional information:

{"message":"Changes must be made through a pull request.","documentation_url":"https://docs.github.com/articles/about-protected-branches"}

@maryliag
Copy link
Contributor Author

bors r+ single on

@craig
Copy link
Contributor

craig bot commented Jun 12, 2023

Build succeeded:

@craig craig bot merged commit 5d630ae into cockroachdb:master Jun 12, 2023
@maryliag maryliag deleted the columns-job branch June 12, 2023 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

db console: high watermark missing from jobs table
3 participants