Skip to content

ui: Add CircleFilled component#75807

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
ericharmeling:circle-filled-icon
Feb 2, 2022
Merged

ui: Add CircleFilled component#75807
craig[bot] merged 1 commit intocockroachdb:masterfrom
ericharmeling:circle-filled-icon

Conversation

@ericharmeling
Copy link
Copy Markdown

Previously, there was no component for a filled circle icon in the ui package.
Recent product designs have requested this icon for the DB Console (see #67510, #73463).
This PR adds a CircleFilled component to the ui package.

Release note: None

@ericharmeling ericharmeling requested a review from gtr February 1, 2022 18:59
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown

@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.

LGTM! Thanks 😄

@ericharmeling ericharmeling requested a review from a team February 1, 2022 19:07
Copy link
Copy Markdown
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained

@ericharmeling
Copy link
Copy Markdown
Author

bors r+

@ericharmeling
Copy link
Copy Markdown
Author

bors r-

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Feb 1, 2022

Canceled.

Previously, there was no component for a filled circle icon in the ui package.
Recent product designs have requested this icon for the DB Console (see cockroachdb#67510, cockroachdb#73463).
This PR adds a CircleFilled component to the ui package.

Release note: None
@ericharmeling
Copy link
Copy Markdown
Author

The Bazel CI keeps failing for this PR. It looks like a timeout that is unrelated to the changes made in this PR.

Any objections to just ignoring the Bazel CI, as it's technically optional, and merging?

@maryliag
Copy link
Copy Markdown
Contributor

maryliag commented Feb 2, 2022

No objections, you can go ahead with the merge

@ericharmeling
Copy link
Copy Markdown
Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Feb 2, 2022

Build succeeded:

@craig craig bot merged commit 63988a4 into cockroachdb:master Feb 2, 2022
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.

4 participants