RI-7811 Show info icon tooltips on hover instead of click #5297
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
Replace RiPopover with RiTooltip for info icons to display on hover instead of requiring a click. Affected components: - CreateRedisearchIndex: Identifier info icon - SubscribeInformation: Pub/Sub channel info icon - RemoveListElements: Redis version limitation info icon References: #RI-7811
pawelangelow
requested review from
ArtemHoruzhenko,
DimoHG,
KrumTy,
dantovska,
pd-redis,
tptodorov and
valkirilov
as code owners
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
redisinsight/ui/src/pages/browser/components/create-redisearch-index/CreateRedisearchIndex.tsx
Show resolved
Hide resolved
pawelangelow
force-pushed
the
fe/bugfix/RI-7811-info-icons-hover
branch
from
68d0d44 to
0c4e834
Compare
Contributor
Code Coverage - Frontend unit tests
Test suite run success5468 tests passing in 703 suites. Report generated by 🧪jest coverage report action from 116c7af |
pd-redis
reviewed
Dec 3, 2025
Contributor
pd-redis
left a comment
pd-redis
left a comment
There was a problem hiding this comment.
I am not sure if it makes sense here, but there is this component in redis-ui:
https://redislabsdev.github.io/redis-ui/?path=/docs/components-moreinfoicon--docs
It definitely goes on the tooltip route.
I however agree with the chatbot, that interactive contents in tooltip can be hard to interact with.
redisinsight/ui/src/pages/browser/components/create-redisearch-index/CreateRedisearchIndex.tsx
Show resolved
Hide resolved
|
|
||
| {!canRemoveMultiple ? ( | ||
| <FlexItem>{InfoBoxPopover()}</FlexItem> | ||
| <FlexItem>{InfoBoxTooltip()}</FlexItem> |
Contributor
There was a problem hiding this comment.
Is this calling a component as a fuinction?!?!?!
| </RiPopover> | ||
| <Text> | ||
| Supported glob-style patterns are described | ||
| <Link |
Tooltips containing clickable links need interactive mode to allow users to hover over the tooltip content and click the links before it disappears. References: #RI-7811
pd-redis
approved these changes
Dec 3, 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.
What
Fix info icons to display tooltips on hover instead of requiring a click.
Replaced
RiPopover(click-triggered) withRiTooltip(hover-triggered) for info icons in:Testing
All info icons should now display their tooltips on hover.
Closes #RI-7811
Pull Request opened by Augment Code with guidance from the PR author
Note
Show info tooltips on hover (RiTooltip) instead of click (RiPopover) in create index, list removal, and pub/sub; update tests accordingly.
RiPopoverwithRiTooltipfor info icons (hover-triggered):browser/components/create-redisearch-index/CreateRedisearchIndex.tsx(Identifier info)browser/modules/key-details/.../RemoveListElements.tsx(Redis version limitation info)pub-sub/components/subscribe-information/SubscribeInformation.tsx(channel/pattern info)pub-sub/components/subscribe-information/SubscribeInformation.spec.tsxto assert tooltip on hover usingwaitForRiTooltipVisible.Written by Cursor Bugbot for commit 116c7af. This will update automatically on new commits. Configure here.