app icon indicating copy to clipboard operation
app copied to clipboard

Feature: Add tooltip to explain PR velocity and overview

Open BekahHW opened this issue 1 year ago • 10 comments

Type of feature

🍕 Feature

Current behavior

Right now, there's not a clear explanation for what PR velocity or overview means.

image of graph with PR velocity and overvies

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

BekahHW avatar Oct 17 '23 16:10 BekahHW

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

dev-phantom avatar Oct 17 '23 19:10 dev-phantom

.take

aryanas159 avatar Oct 18 '23 04:10 aryanas159

The issue is still being triaged. Once the needs triage label has been removed, it can be assigned.

nickytonline avatar Oct 18 '23 12:10 nickytonline

The issue is still being triaged. Once the needs triage label has been removed, it can be assigned.

okay.

aryanas159 avatar Oct 18 '23 13:10 aryanas159

@OgDev-01 were you working on this in a different issue?

BekahHW avatar Nov 15 '23 18:11 BekahHW

@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.

OgDev-01 avatar Nov 16 '23 16:11 OgDev-01

@isabensusan do we have designs for this?

BekahHW avatar May 07 '24 15:05 BekahHW

We can use tooltips like we do in repo pages: image

BekahHW avatar Jul 19 '24 15:07 BekahHW

.take

SURAJ-SHARMA27 avatar Aug 18 '24 04:08 SURAJ-SHARMA27

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?

screen-capture (27).webm

SURAJ-SHARMA27 avatar Aug 18 '24 04:08 SURAJ-SHARMA27