Skip to content

added alert primitive and used it in components#239

Merged
nassery318 merged 3 commits intomainfrom
add-alert-primitive
Mar 10, 2026
Merged

added alert primitive and used it in components#239
nassery318 merged 3 commits intomainfrom
add-alert-primitive

Conversation

@nassery318
Copy link
Contributor

Description

Include a summary of the change.

  1. Added alert primitive component and reused across the app for consistency
  2. Other ui improvements like tooltips, search and etc..

Change Visualization

Include a screenshot/video of before and after the change.

Screen.Recording.2026-03-10.at.2.09.14.PM.mov

Signed-off-by: nassery318 <nassery318@gmail.com>
Signed-off-by: nassery318 <nassery318@gmail.com>
label="Total Keys"
tooltip={
<TooltipIcon description="Total number of keys in the database" size={14} />
<TooltipIcon description="Total number of keys scanned/fetched" size={14} />
Copy link
Contributor

Choose a reason for hiding this comment

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

Would "sampled" be a better word here than "scanned/fetched"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Definitely, sampled is much better. Thanks

Signed-off-by: nassery318 <nassery318@gmail.com>
@nassery318 nassery318 requested a review from ArgusLi March 10, 2026 19:15
@nassery318 nassery318 merged commit 672ec6c into main Mar 10, 2026
7 checks passed
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