Skip to content

Redesign Authors & Maintainers section with ranked leaderboard UI#491

Open
Ajaypanchal4326 wants to merge 1 commit intoNodeSecure:masterfrom
Ajaypanchal4326:master
Open

Redesign Authors & Maintainers section with ranked leaderboard UI#491
Ajaypanchal4326 wants to merge 1 commit intoNodeSecure:masterfrom
Ajaypanchal4326:master

Conversation

@Ajaypanchal4326
Copy link
Contributor

✨ UI Improvement: Authors & Maintainers

This PR modernizes the Authors & Maintainers section to make it more
informative, readable, and useful in both HTML and PDF reports.

🔍 Problem

  • The previous avatar-only layout provided limited value, especially in PDF output.
  • There was no clear way to identify top contributors or understand relative impact.

✅ Solution

This PR introduces a ranked leaderboard-style layout:

  • Authors are sorted by occurrence count (descending)
  • Clear ranking (1, 2, 3, …) based on contribution frequency
  • Top contributors are visually highlighted
  • Avatars are preserved for visual context
  • No JavaScript added, no data model changes

🎯 Key Improvements

  • 🏆 Ranked leaderboard for quick contributor comparison
  • 📄 Improved PDF readability (list-based, no hover dependency)
  • 🎨 Modern, clean UI with clear hierarchy
  • 🔒 Presentation-only change (HTML + CSS)

🧠 Ranking Logic

Authors are ranked by their occurrence count in dependencies:

  • Higher count ⇒ higher rank
  • Sorting is handled at the template level for presentation clarity

🖼️ Preview

Report_git

📁 Files Changed

  • views/template.html
  • public/css/style.css

🧪 Testing

  • npm run build
  • Verified in HTML preview
  • Verified PDF output

Feedback and suggestions are welcome 🙌

@PierreDemailly
Copy link
Member

Hi @Ajaypanchal4326, sorry for the late response

I tested on a package with ~50 contributors and the authors leaderboard already spans 2 pages 😬 And we can easily have way more than 50.

Also, clicking on an author opens a mailto: link, which is pointless here, we don't need to email them.

That said, one thing I do like about these changes is that we can now see the total number of contributors at a glance, that information wasn't surfaced anywhere before.

Maybe we could display the total author count, but only show a subset, like contributors with more than one contribution, and/or just the top 10-15. What do you think?

Copy link
Member

@PierreDemailly PierreDemailly left a comment

Choose a reason for hiding this comment

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

lgtm % nits

@Ajaypanchal4326
Copy link
Contributor Author

Thanks for the review and suggestions
I've applied the requested changes
Let me know if anything else needs adjustment!

@Ajaypanchal4326
Copy link
Contributor Author

Let me know if anything else needs adjustment!

Copy link
Member

@PierreDemailly PierreDemailly left a comment

Choose a reason for hiding this comment

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

lgtm, don't hesitate to squash your commits 👍

@Ajaypanchal4326
Copy link
Contributor Author

Thanks for the suggestion!
I've squashed the commits to keep the history clean.

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.

3 participants