-
Notifications
You must be signed in to change notification settings - Fork 404
RI-7598 Change visuals of the "Import sample data" popover #5036
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
Conversation
/> | ||
<div className={styles.popoverFooter}> | ||
<div> | ||
<Row gap="m" justify="end"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
<div className={styles.confirmDialogActions}> | ||
<Spacer size="l" /> | ||
<Row justify="end" gap="m"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Coverage - Frontend unit tests
Test suite run success5162 tests passing in 677 suites. Report generated by 🧪jest coverage report action from 8fa8548 |
</FormField> | ||
</FlexItem> | ||
|
||
{!canRemoveMultiple ? ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bd2101f
to
8fa8548
Compare
{notification.title} | ||
</Title> | ||
|
||
<Spacer size="s" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
} | ||
|
||
.popoverLikeTooltip { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the actual change related to this PR. Sorry for the other minor fixes and refactoring.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well done! thank you
Description
Update the colors of the "Import sample data" popover to follow the color palette defined by Redis UI.
a.k.a remove a custom color override introduced for EUI 4 years ago.
How it was tested
Note: Make sure that you have a clean database. If not, you can run flushdb in the CLI.
Note
Also, this is a global change, so it does affect all the popovers in the application that was using the blue color. Now, they all should follow the Redis UI color palette.
Tutorials
And you can run any of the code snippets under the "EXACT MATCH" section
Tips
RDI
JSON key override
Delete multiple List keys
Note: This info popover is visible only on Redis versions less than 6.2
It will open a form at the bottom of the page, with an info icon + popover
Create Redisearch index
Redis Copilot
generalChatAgreements
in the local storage)Notifications Center