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: ux improvements #92765

Merged
merged 1 commit into from
Nov 30, 2022
Merged

ui: ux improvements #92765

merged 1 commit into from
Nov 30, 2022

Conversation

maryliag
Copy link
Contributor

This commit adds UX improvements, such as:

  • Add spacing on Table Details page
    Before

Screen Shot 2022-11-30 at 2 24 03 PM

After
Screen Shot 2022-11-30 at 2 36 29 PM

  • Add spacing on Index Details page and change the sql box size
    Before

Screen Shot 2022-11-30 at 2 39 09 PM

After
Screen Shot 2022-11-30 at 2 41 56 PM

  • Rename Statement Execution and Planning Time chart on Statement Details page to Statement Times
    Before

Screen Shot 2022-11-30 at 2 50 34 PM

After
Screen Shot 2022-11-30 at 2 50 01 PM

Epic: None

Release note (ui change): Rename chart on Statement Details page from Statement Execution and Planning Time to Statement Times.

@maryliag maryliag requested review from dongniwang and a team November 30, 2022 19:52
@cockroach-teamcity
Copy link
Member

This change is Reviewable

This commit adds UX improvements, such as:
- Add spacing on Table Details page
- Add spacing on Index Details page and change
the sql box size
- Rename `Statement Execution and Planning Time` chart
on Statement Details page to `Statement Times`

Epic: None

Release note (ui change): Rename chart on Statement Details
page from `Statement Execution and Planning Time` to
`Statement Times`.
@maryliag
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 30, 2022

Build succeeded:

@craig craig bot merged commit 7e04319 into cockroachdb:master Nov 30, 2022
@maryliag maryliag deleted the update-title branch November 30, 2022 23:34
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.

3 participants