app
app copied to clipboard
Feature: Add tooltip to explain PR velocity and overview
Type of feature
🍕 Feature
Current behavior
Right now, there's not a clear explanation for what PR velocity or overview means.
Initially raised in docs, we feel like that's better to handle it programmatically on the app side of things.
Suggested solution
- Add a tooltip on hover over PR velocity and PR overview that explains those terms.
Additional context
No response
Code of Conduct
- [X] I agree to follow this project's Code of Conduct
Contributing Docs
- [X] I agree to follow this project's Contribution Docs
Hey there! I've got a suggestion for improving the user experience. Instead of using tooltips, I recommend incorporating an "info" icon next to the respective text, like "PR velocity" and then the info icon, and so on. This will make it more user-friendly, especially on mobile devices where tooltips can be challenging. It provides a clear way for users to access additional information without the need for hovering
.take
The issue is still being triaged. Once the needs triage
label has been removed, it can be assigned.
The issue is still being triaged. Once the
needs triage
label has been removed, it can be assigned.
okay.
@OgDev-01 were you working on this in a different issue?
@OgDev-01 were you working on this in a different issue?
not at all, but i will pick this up later today and get something in as a POC with our regular tooltip for now. discussion can continue in the PR.
@isabensusan do we have designs for this?
We can use tooltips like we do in repo pages:
.take
Hi @nickytonline @BekahHW, I have reused the above mentioned tooltip for both the PR Overview and Velocity. Will this work, or are any changes needed. Should I proceed for a PR?