Skip to content

Conversation

@oalders
Copy link
Member

@oalders oalders commented May 3, 2025

  • Add GitHub stars to favorite.tx

This can be tested via the url /release/OALDERS/HTML-Restrict-v3.0.2.

Screenshot 2025-05-03 at 1 10 02 PM

Note how badly styled it is. There's no need to keep anything from the current style.

@codecov
Copy link

codecov bot commented May 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.46%. Comparing base (1294326) to head (0d3fc52).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3341      +/-   ##
==========================================
+ Coverage   73.45%   73.46%   +0.01%     
==========================================
  Files          68       68              
  Lines        2388     2389       +1     
  Branches      335      335              
==========================================
+ Hits         1754     1755       +1     
  Misses        508      508              
  Partials      126      126              
Files with missing lines Coverage Δ
lib/MetaCPAN/Web/Model/ReleaseInfo.pm 82.05% <100.00%> (+0.15%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@oalders oalders force-pushed the oalders/stars branch 3 times, most recently from 43b4926 to 7b70e4c Compare May 3, 2025 17:18
<div class="show-logged-out">
<button class="fav-not-logged-in favorite[% if $favorites { %] highlight[% } %]"><span>[% $favorites %]</span> ++</button>
</div>
%% if $stars {
Copy link

Choose a reason for hiding this comment

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

Should the control disappear when there are zero stars?

Copy link
Member Author

Choose a reason for hiding this comment

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

That was my original thought, but now I'm not sure.

@JordanEdward02
Copy link

Tried to make it match the favourites button as much as I could, but obviously couldn't go with orange background because the contrast with the star is awful. Please let me know if you think any changes are needed :)

Screenshot from 2025-05-06 17-54-07
Screenshot from 2025-05-06 17-54-01

@JordanEdward02 JordanEdward02 marked this pull request as ready for review May 6, 2025 17:08
@guest20
Copy link

guest20 commented May 6, 2025

Perhaps there should be a github icon in the area to indicate that they are github stars?

Also, when talking about github stars the yellow/full star means you, yourself have starred it:
image

This is what the control on github looks like when you haven't starred the thing yourself:
image

@oalders
Copy link
Member Author

oalders commented May 11, 2025

Tried to make it match the favourites button as much as I could, but obviously couldn't go with orange background because the contrast with the star is awful.

Thanks for this! For some reason I missed the notification until now.

Perhaps there should be a github icon in the area to indicate that they are github stars?

I think there is a font-awesome GitHub icon we could add to this.

Also, when talking about github stars the yellow/full star means you, yourself have starred it

We don't have a good way of knowing if the user has starred the repo or not, so I don't know that there is a perfect solution here.

@oalders oalders merged commit f3a745e into master May 11, 2025
11 checks passed
@oalders oalders deleted the oalders/stars branch May 11, 2025 00:53
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.

4 participants