-
-
Notifications
You must be signed in to change notification settings - Fork 480
[UX] Add Guided tour for Library, filters and Sidebar #4401
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
…nd Tour components
…r for guided tour
… step descriptions
…oicVersion CSS file
…d enhanced descriptions
public/locales/en/translation.json
Outdated
"title": "Enjoy your games!" | ||
}, | ||
"filters": "Filter your games by store, platform, favorites and more using these options.", | ||
"gameCard": "Right Click on thegame card to navigate to the game page and to see details and adjust settings or Left click to open the context menu.", |
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.
These are backwards, left click opens the game page, right click opens the context menu
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.
also, when there are no games, this step is still displayed, but doesn't have a card to point
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.
fixed
src/frontend/components/UI/Sidebar/components/HeroicVersion/index.tsx
Outdated
Show resolved
Hide resolved
'info.heroic.click-to-see-changelog', | ||
'Click to see changelog' | ||
<div className="heroicVersionContainer" data-tour="sidebar-version"> | ||
<> |
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.
we don't need this fragment if we are adding the wrapper div, it should make the diff less complicated cause it won't change the indentation of everything inside
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.
removed
This adds two new help icons that will trigger a small guided tour, one for the library, gamecard and filters and another one for the Sidebar.
Gravar.a.tela_20250307_224042.mp4
Gravar.a.tela_20250307_221356.mp4
AI Summary
This pull request introduces a new guided tour feature using
intro.js
andintro.js-react
to help users navigate the application. The changes include adding new dependencies, updating localization files, and creating new components for the tour functionality. Additionally, custom styles have been added to ensure the tour elements match the application's design.New Guided Tour Feature:
Dependencies Added:
intro.js
andintro.js-react
topackage.json
andpnpm-lock.yaml
to support the guided tour feature. [1] [2] [3] [4]Localization Updates:
public/locales/en/translation.json
to include new strings for the tour feature.New Components:
TourProvider
tosrc/frontend/App.tsx
to wrap the application and manage tour state. [1] [2] [3]Tour
component insrc/frontend/components/Tour/Tour.tsx
to handle the steps and options of the guided tour.TourButton
component insrc/frontend/components/Tour/TourButton.tsx
to trigger the guided tour.Custom Styles:
src/frontend/components/Tour/Tour.scss
andsrc/frontend/components/Tour/TourButton.scss
to match the application's design. [1] [2]Integration with Existing UI:
ActionIcons
,CategoryFilter
,LibraryFilters
, andLibrarySearchBar
to guide users through different parts of the application. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]These changes collectively enhance the user experience by providing an interactive guide to navigate and understand the application features.
Use the following Checklist if you have changed something on the Backend or Frontend: