Skip to content
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

Feature: add explanation of what users are seeing on list activity graph #2270

Open
2 tasks done
BekahHW opened this issue Dec 6, 2023 · 11 comments · May be fixed by #3328
Open
2 tasks done

Feature: add explanation of what users are seeing on list activity graph #2270

BekahHW opened this issue Dec 6, 2023 · 11 comments · May be fixed by #3328
Assignees
Labels

Comments

@BekahHW
Copy link
Member

BekahHW commented Dec 6, 2023

Type of feature

🍕 Feature

Current behavior

It's not clear what the chart is representing.

current activity chart

Suggested solution

  1. Add subtext below like we do for the Most Active Contributors chart

added subtext

You should be able to just add this below the heading:

<div class="text-sm text-slate-400 mb-4">Based on the number of PRs
</div>

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
Copy link

github-actions bot commented Dec 6, 2023

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

@nickytonline
Copy link
Member

I wouldn't go with a tooltip for this as it's important information we should surface instead of them having to discover they have to hover over the title to find out what it is.

Does that work for you @isabensusan?

@CBID2
Copy link
Contributor

CBID2 commented Dec 13, 2023

I wouldn't go with a tooltip for this as it's important information we should surface instead of them having to discover they have to hover over the title to find out what it is.

Does that work for you @isabensusan?

Yeah I agree with @nickytonline. Subtexts would make it easier for users to understand what's going on.

@manipandit
Copy link
Contributor

.take

@BekahHW
Copy link
Member Author

BekahHW commented Apr 4, 2024

@manipandit how are things going?

@devharsh2k4
Copy link

devharsh2k4 commented May 6, 2024

Hey looks like there isn't any activity so can i get assigned

@manipandit
Copy link
Contributor

@BekahHW I would like to discontinue working on this issue. You can assign it to others..

@devharsh2k4
Copy link

.take

@devharsh2k4
Copy link

@BekahHW , @nickytonline Please review my PR for this feature

@nickytonline
Copy link
Member

@BekahHW , @nickytonline Please review my PR for this feature

@devharsh2k4, there's no need to ask us to review your PR. We all receive notifications about activity in the repository.

@devharsh2k4
Copy link

Ok got it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment