-
Notifications
You must be signed in to change notification settings - Fork 407
Fe/feature/ri 7039 replace eui #4819
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
Closed
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # redisinsight/ui/src/components/connectivity-error/ConnectivityError.tsx # redisinsight/ui/src/components/inline-item-editor/InlineItemEditor.tsx # redisinsight/ui/src/components/item-list/components/delete-action/DeleteAction.tsx # redisinsight/ui/src/components/item-list/components/export-action/ExportAction.tsx # redisinsight/ui/src/components/query/query-actions/QueryActions.tsx # redisinsight/ui/src/components/query/query-card/QueryCardHeader/QueryCardHeader.tsx # redisinsight/ui/src/components/side-panels/panels/enablement-area/EnablementArea/components/InternalPage/InternalPage.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeyHash/AddKeyHash.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeyList/AddKeyList.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeyReJSON/AddKeyReJSON.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeySet/AddKeySet.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeyStream/AddKeyStream.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeyString/AddKeyString.tsx # redisinsight/ui/src/pages/browser/components/add-key/AddKeyZset/AddKeyZset.tsx # redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/components/edit-entire-item-action/EditEntireItemAction.tsx # redisinsight/ui/src/pages/browser/modules/key-details/components/set-details/add-set-members/AddSetMembers.tsx # redisinsight/ui/src/pages/database-analysis/components/header/Header.tsx # redisinsight/ui/src/pages/home/components/database-list-header/DatabaseListHeader.tsx # redisinsight/ui/src/pages/home/components/database-manage-tags-modal/ManageTagsModal.tsx # redisinsight/ui/src/pages/settings/components/cloud-settings/CloudSettings.tsx # redisinsight/ui/src/pages/settings/components/cloud-settings/components/user-api-keys-table/UserApiKeysTable.tsx # redisinsight/ui/src/pages/slow-log/components/Actions/Actions.tsx
* import the modal components * apply the form dialog; vefiry for the database connection form; delete unneeded styles * replace filter key type modal * remove filter key type eui styles * replace browser search panel / module not loaded * replace consents settings popup modal * replace rdi import modal(s) * add ids to buttons * replace the start rdi pipeline modal * remove background colors * remove :global styling * remove the advantages background * replace select account dialog modal * replace select plan modal * replace oauth sso dialog modal * finalize form dialog and manage tags modal * round up the width, set too 601 for no reason (: * format form dialog file * use width modal prop instead of as part of the class * fix the rebase issues - use the propper RI file picker * and fix the upload warning container * refactor size capital to small letter; format * change rdi error configuration file icon and add color; align items * return the exported type and format file * use cx when multiple classes applied * fix some of the tests failing due to issues with the modal header; skip the ones that are tied directly to the header as the header is set to null * fix upload dialog tests * fix database panel dialog * change oauth sso dialog and browser search panel dialog to null * fix width * remove duplicate styling * remove not needed code * fix oauth select account dialog * fix all tests regarding the buggy modal title * reverting changes to the features config.json --------- Co-authored-by: Kristiyan Ivanov <[email protected]>
…d the implementation around the main content under the new navigation (#4784) * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - added border above the new navigation * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - updated main content brackgrounds to use the proper colors * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - added borders to the keys list and key details to mimic the new DS * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - removed background for the bottom buttons (add, cancel, save, etc). In the new DS there is no different BG and it only makes it mroe complciated to keep track * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - add padding on top of the search bar to distance it from the new navigation * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - updated the cli row to look like the previews * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - updated spacing under the navigation to be handled in one place * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - fixed paddings for the CLI row * RI-7269 - Fix major discrepancies between the DS previews and the implementation around the main content under the new navigation - Updated uses of theme to be through useTheme, instead of a direct import
* create loading logo component * move import from layout to display * add sizing to the component * use $ for size and bounceSpeed instead of Omit * implement redis theme sizing * Rename folder logo-loading -> loading-logo * create and export RiEmptyPrompt component * replace the eui component with the new one * Include the ...rest in the empty prompt component * Extended spacer to support theme sizes --------- Co-authored-by: Kristiyan Ivanov <[email protected]>
* RI-7244 -Edit-and-remove-buttons-are-not-aligned * Update redisinsight/ui/src/components/base/forms/buttons/EmptyButton.tsx Co-authored-by: pd-redis <[email protected]> * Update redisinsight/ui/src/components/base/forms/buttons/EmptyButton.tsx Co-authored-by: pd-redis <[email protected]> --------- Co-authored-by: pd-redis <[email protected]>
* provide actual icon element to IconButton * set LikeIcon as default for VoteOption.tsx
* reverse link behavior
* RI-7252 - RDI---Broken-layout-for-the-list-of-jobs - added size props for the inline item editor's action section * RI-7252 - RDI---Broken-layout-for-the-list-of-jobs - added size props for the inline item editor's action section
* RI-7279 - "test" element is visible on top of the db keys list * RI-7278 - Remove the old navigation for DBs * RI-7278 - Remove the old navigation for DBs - removed unused variables and updated tests
* RI-7217 allow to deplot rdi pipeline with validation errors * align text and icon * RI-7217 move errors list to a separate component and add tests * RI-7217 fix PR comments * RI-7217 fix test --------- Co-authored-by: pd-redis <[email protected]>
* RI-7186 validate job name separately * RI-7186 add tests
KIvanow
approved these changes
Aug 7, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.