ui: update Hot ranges page styles#79133
Conversation
|
This change is |
17adffb to
9b9d660
Compare
Santamaura
left a comment
There was a problem hiding this comment.
Reviewed 2 of 3 files at r1, all commit messages.
Reviewable status:complete! 0 of 0 LGTMs obtained (waiting on @koorosh and @zachlite)
pkg/ui/workspaces/db-console/src/views/hotRanges/index.tsx, line 68 at r1 (raw file):
(queries per second). <br /> Use this information to <Anchor href="" target="_blank">
Are we removing the link here on purpose?
Current change introduces several minor UI changes on Hot Ranges page: - changed page description, not it isn't rendered as inline alert; - Data formatting is changed to use UTC; - Changed heading style to H3 to follow the same style as other pages; - QPS values are rounded to 2 decimals; Release note (ui change): minor styling changes on Hot Ranges page to follow the same style as other pages.
9b9d660 to
488d64b
Compare
koorosh
left a comment
There was a problem hiding this comment.
Reviewable status:
pkg/ui/workspaces/db-console/src/views/hotRanges/index.tsx, line 68 at r1 (raw file):
Previously, Santamaura (Alex Santamaura) wrote…
Are we removing the link here on purpose?
oh, that's my fault, it should be brought back. Thanks!
Santamaura
left a comment
There was a problem hiding this comment.
Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status:
a discussion (no related file):
|
bors r+ |
|
Build succeeded: |
Current change introduces several minor UI changes
on Hot Ranges page:
inline alert;
as other pages;
Release note (ui change): minor styling changes on
Hot Ranges page to follow the same style as other pages.
Resolves: #78517
Screens:
Jira issue: CRDB-14827