Skip to content

Conversation

@valkirilov
Copy link
Member

@valkirilov valkirilov commented Oct 20, 2025

Description

Fix the Onboarding wizard, showcasing the basic features provided by Redis Insight

  • fix the placement of the start step, move it to the right

Before EUI replacement

Screen.Recording.2025-10-20.at.11.05.53.mov

After the changes introduced in this pull request

Screen.Recording.2025-10-21.at.15.32.53.mov

How it was tested

  1. Go to Databases and open a connection to an existing database instance, or create a new one
  2. Go to Browser page
  3. From the left side nav, open the Help menu item, and from the popover click on Reset Onboarding button

The popover with the onboarding wizard should appear on the screen, and you can follow the steps.

Start and finish steps step

Fixed the placement of the popover on the first and the last step of the onboarding wizard.

Old Before After
image image image
image image image

Change the position of the popovers on the navbar

Previously, when the navigation was in the left sidenav, all popovers were positioned next to it, with an arrow pointing to the left. Now, when the navigation is at the top, there is no point in showing the popovers next to it.

Instead, I have moved them below it, and updated the arrows pointing upwards.

Before After
image 2025-10-21_13-46

Change the styling of the links

Updated the links in the onboarding popovers to use Redis UI instead of what we had before.

Before After
image image
image image

- removed sass styles and replaced with flex items
- fix the position of the popover

re #RI-7627
@valkirilov valkirilov self-assigned this Oct 20, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 21, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.2% 20057/24400
🟡 Branches 67.56% 8683/12852
🟡 Functions 76.34% 5340/6995
🟢 Lines 82.62% 19635/23766

Test suite run success

5162 tests passing in 677 suites.

Report generated by 🧪jest coverage report action from a65ea1d

@valkirilov valkirilov marked this pull request as ready for review October 21, 2025 13:38
- added onboarding popovers to the main app navigation
- change the position of the popover in the navbar, to show bellow it, instead of next to it
- change the styles of the links in the onboarding popovers, to follow Redis UI styles
- update the position of the last step of the onboarding flow
- replace sass styles with layout components
- cleanup leftover sass styles

re #RI-7624
@valkirilov valkirilov force-pushed the fe/bugfix/RI-7627/onboarding branch from 3dba6f5 to a65ea1d Compare October 22, 2025 07:58
@valkirilov valkirilov merged commit bd51632 into main Oct 27, 2025
18 checks passed
@valkirilov valkirilov deleted the fe/bugfix/RI-7627/onboarding branch October 27, 2025 11:11
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