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
Added Tooltip to Susi Launcher Button #1770 #1771
Conversation
MUI lib provides tooltips on buttons. See if you can make use of them. |
@AakashMallik Actually, we are using self-made button and the tooltip in material UI requires the use of the material button to add that particular tooltip. So I have to add my own tooltip. |
@sk9331657 No one is stopping you from using their buttons btw. You are just using your own svg icon, which shall be no problem as far as I know. |
@AakashMallik I will surely check if that is possible without breaking styling of button. |
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 are already using tooltip, search for tooltip in package.json and use that
bb9c256
to
e620e1d
Compare
@fragm3 Done that way. |
@Akshat-Jain @akshatnitd Please Review. |
Good work @sk9331657. But I am not sure if this is required. Its already quite intuitive to use the launcher. |
Fixes #1770
Changes: Added Tooltip to Susi Launcher Button #1770
Surge Deployment Link: https://pr-1771-fossasia-susi-skill-cms.surge.sh
Screenshots for the change: