anuraghazra / github-readme-stats

:zap: Dynamically generated stats for your github readmes

Home Page:https://github-readme-stats.vercel.app

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add an option to display byte count for Top-languages

abap34 opened this issue · comments

Is your feature request related to a problem? Please describe.

Currently, the Top Language Card only displays the percentage of each language used. However, adding an option to display byte counts could incentivize more coding activity, thereby increasing the amount of code generated by individuals.

Describe the solution you'd like

I have already implemented this on my self-hosted github-readme-stats.
( So, if there is demand for this feature from the community, I can clean up the implementation and submit a pull request :)

This can be achieved by making minor modifications to the api/top-lang.js and src/cards/top-languages-card.js.

Describe alternatives you've considered

No response

Additional context

It might require CSS adjustments for many layouts. For instance, the default layout appears as follows:

Hey, @abap34! Thanks for your interest in our project and submitting feature request. You're right, we use top issues dashboard to prioritize and track community demand, if this feature will be supported by the community feel free to open a pull request for me to review.

Thank you for your response @qwerty541!
I'll submit a pull request if it gets featured on the dashboard 😄

Just want to additionally note that we can avoid neccesity of making changes in layout by shorter formatting of bytes numbers: 670,404 bytes => 670k bytes. Also we can convert bytes in KB, MB and GB when it will be enough for one unit: 670,404 bytes => 670KB.

That idea sounds very good!
Given the current number of characters in the percentages, I think a format like "670 KB" would be better.

This issue seems to be featured on the dashboard, so I will implement it and send a pull request.

I want to warn you in advance that review and merging of your pull request can be postponed for several months. We have rule in our team that merging features require approval of at least 2 members. Currently I'm alone active, other members busy, so don't hurry with opening pull request.

Oh, I see. Thank you for letting me know.
Sorry, I've already submitted the pull request, so please review it whenever you have the time.
I appreciate your daily maintenance work!

Thanks for your efforts implementing this feature and for your kind words. I just left a review #3708 (review).