ui: improve tooltip UX with text updates#83420
Conversation
895d5da to
3495017
Compare
maryliag
left a comment
There was a problem hiding this comment.
Smaller nit on your commit message, otherwise
Reviewable status:
complete! 1 of 0 LGTMs obtained (waiting on @ericharmeling and @maryliag)
-- commits line 3 at r1:
nit: your commit message should also have the "Fixes..."
your release note should state there were updates on tooltips
3495017 to
f7f1eab
Compare
ericharmeling
left a comment
There was a problem hiding this comment.
Reviewable status:
complete! 0 of 0 LGTMs obtained (and 1 stale) (waiting on @maryliag)
Previously, maryliag (Marylia Gutierrez) wrote…
nit: your commit message should also have the "Fixes..."
your release note should state there were updates on tooltips
Fixed!
f7f1eab to
777b2e6
Compare
Fixes cockroachdb#81374. Fixes cockroachdb#83256. Fixes cockroachdb#81248. Fixes cockroachdb#79018. Release note (ui change): Updated tooltips on the Statements and Transactions pages in the DB Console for improved UX.
777b2e6 to
b3571fe
Compare
|
bors r+ |
|
Build succeeded: |
|
@ericharmeling, @maryliag I'm reverting date range to time interval throughout in the docs. If we no longer want to use date range, shouldn't this UX label be updated as well? See filed issue #84361 |
I believe so. Thank you for pointing that out and for filing an issue! |
Fixes #81374.
Fixes #83256.
Fixes #81248.
Fixes #79018.
Note the following:
Release note (ui change): Updated tooltips on the Statements and
Transactions pages in the DB Console for improved UX.