studio
studio copied to clipboard
feat: added tooltips for navigation bar (#193)
Description
Following #193, I added some tooltips to the navigation bar.
Let me know what you think @mcturco @magicmatatjahu
✔️ Deploy Preview for modest-rosalind-098b67 ready!
🔨 Explore the source changes: abab3c0c6e594e20b642da7735f78af323eead82
🔍 Inspect the deploy log: https://app.netlify.com/sites/modest-rosalind-098b67/deploys/61bb1808fc8012000753160a
😎 Browse the preview: https://deploy-preview-215--modest-rosalind-098b67.netlify.app/
Kudos, SonarCloud Quality Gate passed! 
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
Thanks @boyney123 for adding this!! 🎉 However, I think if we made the background of the labels fully opaque it would look a lot cleaner. I think it might be a little distracting to see things peeking through behind the labels as the text size is already rather small. What do you think?
It should be integrated not with icon, but with whole nav item :)
I agree with @magicmatatjahu here completely!! Better accessibility 😎
@boyney123 @magicmatatjahu
Hey guys! Was cleaning out my inbox and realized that this PR has yet to be merged. I think this is still a great feature that we should consider revisiting and merging. I think all we should do is make the tool tips fully opaque instead of transparent and integrate them with the entire button and not just the icon. Then we should definitely ship it! What do you guys think?
@mcturco Yeah, we should handle that PR as you wrote. If @boyney123 you don't have time I can handle that, no problem :)
This pull request has been automatically marked as stale because it has not had recent activity :sleeping:
It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.
There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.
Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.
Thank you for your patience :heart:
This pull request has been automatically marked as stale because it has not had recent activity :sleeping:
It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.
There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.
Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.
Thank you for your patience :heart:
Tooltips added in the https://github.com/asyncapi/studio/pull/434 PR
@boyney123 I hope that you don't have a problem that I'm closing this PR :)