Skip to content

Remove upvote counter from landing-page cards#495

Merged
soyalejolopez merged 1 commit into
microsoft:masterfrom
soyalejolopez:alejanl-microsoft-remove-card-votes
Jul 22, 2026
Merged

Remove upvote counter from landing-page cards#495
soyalejolopez merged 1 commit into
microsoft:masterfrom
soyalejolopez:alejanl-microsoft-remove-card-votes

Conversation

@soyalejolopez

Copy link
Copy Markdown
Member

What & why

The card upvote count was baked into a static resource-stats.json that is only regenerated by a daily CI job (cron: 0 6 * * *). A fresh reaction wouldn't surface on the cards for up to ~24h, and today only one resource has a backing discussion, so the counter read as stale/misleading and looked like something you could vote on directly from the card (you can't).

What this changes

  • Removes the upvote count display from both the comfortable and compact card views
  • Removes the compact Votes header column and the orphaned .compact-votes / .ch-votes CSS

What is intentionally preserved

  • Voting still works on the detail page via the giscus-backed Community & upvotes panel
  • upvotes is still loaded in getStats and still drives the Popular / Trending / Engaging sort logic

Display-only removal — no data or sort behavior changes.

The card upvote count was baked into a static resource-stats.json that is
only regenerated by a daily CI job (cron 0 6 * * *), so a fresh reaction
would not show up for up to ~24h. Combined with the fact that only one
resource currently has a backing discussion, the counter was misleading.

This removes the upvote count *display* from both the comfortable and
compact card views (and the compact 'Votes' header column + orphaned CSS).
Voting itself is unchanged: the giscus-backed 'Community & upvotes' panel
on the detail page still works, and the upvotes value is still loaded in
getStats and used by the Popular/Trending/Engaging sort logic.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 888b1353-6309-4a90-bb0f-9b7b716a32dc
@soyalejolopez
soyalejolopez merged commit 770a33a into microsoft:master Jul 22, 2026
2 checks passed
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.

1 participant