Skip to content

Added medal colors for top-3 badges#100

Merged
ashkulz merged 3 commits into
ashkulz:masterfrom
kudima03:add_colors_for_top_3
Jan 6, 2026
Merged

Added medal colors for top-3 badges#100
ashkulz merged 3 commits into
ashkulz:masterfrom
kudima03:add_colors_for_top_3

Conversation

@kudima03

Copy link
Copy Markdown
Contributor

Improved GitHub ranking badges with a new medal-style color scheme:

  • 1st — Gold
  • 2nd — Silver
  • 3rd — Bronze
  • Other ranked — Blue
  • Unranked — Red

This helps highlight top contributors and improves badge readability.

@kudima03

Copy link
Copy Markdown
Contributor Author

Hi @ashkulz! Check when you have a minute.

@kudima03

Copy link
Copy Markdown
Contributor Author

Hi @ashkulz! Reminder.

@ashkulz ashkulz left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, was going to leave feedback earlier but had health issues in the family.

Comment thread badges/cloudflare_worker.js Outdated
@ashkulz

ashkulz commented Nov 28, 2025

Copy link
Copy Markdown
Owner

I'm also not convinced that the colors look great or make sense. I'd rather add a medal in the text itself instead of # e.g. https://img.shields.io/badge/any_text-you_like%F0%9F%A5%87-FF0000

Co-authored-by: Ashish Kulkarni <ashish@kulkarni.dev>
@kudima03

Copy link
Copy Markdown
Contributor Author

@ashkulz In my opinion medal colors enhance the visual appearance of the badges, creating a sense of exclusivity and achievement. Medal in text is little and almost invisible.

Wishing your family good health - take care of them and yourself.

@kudima03 kudima03 requested a review from ashkulz November 28, 2025 13:07
@kudima03

kudima03 commented Dec 8, 2025

Copy link
Copy Markdown
Contributor Author

Hi @ashkulz! Reminder.

3 similar comments
@kudima03

Copy link
Copy Markdown
Contributor Author

Hi @ashkulz! Reminder.

@kudima03

Copy link
Copy Markdown
Contributor Author

Hi @ashkulz! Reminder.

@kudima03

kudima03 commented Jan 6, 2026

Copy link
Copy Markdown
Contributor Author

Hi @ashkulz! Reminder.

@ashkulz ashkulz merged commit 719e552 into ashkulz:master Jan 6, 2026
1 check passed
@ashkulz

ashkulz commented Jan 6, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution, @kudima03! And apologies for the delay in getting this merged.

@kudima03

kudima03 commented Jan 6, 2026

Copy link
Copy Markdown
Contributor Author

@ashkulz Thank you for the review!

@kudima03 kudima03 deleted the add_colors_for_top_3 branch January 6, 2026 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants