Skip to content

docs: prefix GitHub star count with ★ glyph#883

Merged
jdx merged 2 commits intomainfrom
docs/star-glyph
Apr 27, 2026
Merged

docs: prefix GitHub star count with ★ glyph#883
jdx merged 2 commits intomainfrom
docs/star-glyph

Conversation

@jdx
Copy link
Copy Markdown
Owner

@jdx jdx commented Apr 26, 2026

Adds a small ★ (U+2605) before the GitHub stargazer count in the top-nav social link, so it reads "★ 784" instead of just "784".


Note

Low Risk
Low risk UI-only change limited to VitePress docs theme DOM injection and CSS styling, with no data, auth, or behavioral impact beyond presentation.

Overview
Updates the VitePress docs theme to render the GitHub star badge as ★ <count> by inserting a new .star-glyph element before the existing star count text.

Adds small CSS rules to style the glyph (font fallback, spacing) while keeping existing hover and responsive behavior unchanged.

Reviewed by Cursor Bugbot for commit 33f68fa. Bugbot is set up for automated code reviews on this repo. Configure here.

Adds a small ★ (U+2605) before the stargazer count in the top-nav
social link, so it reads "★ 1.2k" instead of just "1.2k".
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 26, 2026

Greptile Summary

This PR adds a decorative ★ (U+2605) glyph before the GitHub star count in the VitePress top-nav social link, so it displays as ★ 784 instead of just 784. The change is docs-only, correctly marks the glyph aria-hidden="true" for accessibility, and scopes the new CSS tightly under .VPSocialLinks.

Confidence Score: 5/5

Safe to merge — pure docs/UI change with no logic or data handling impact.

Both changed files are confined to the VitePress docs theme. The DOM change is minimal and well-guarded, accessibility is handled correctly with aria-hidden, and the CSS is tightly scoped. No functional code is affected.

No files require special attention.

Important Files Changed

Filename Overview
docs/.vitepress/theme/index.ts Adds a decorative ★ glyph span (aria-hidden) before the star count text node; clean DOM construction with no logic changes.
docs/.vitepress/theme/style.css Adds scoped CSS for .star-glyph with font-family, line-height, and margin-right; no regressions to existing rules.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[addStarCount called] --> B{starsData.stars truthy?}
    B -- No --> C[return false]
    B -- Yes --> D[querySelectorAll GitHub nav links]
    D --> E{.star-count already present?}
    E -- Yes --> F[skip link]
    E -- No --> G[create span.star-count]
    G --> H[create span.star-glyph ★ aria-hidden=true]
    H --> I[append glyph + starsData.stars text node]
    I --> J[appendChild starBadge to githubLink]
Loading

Reviews (2): Last reviewed commit: "docs: render star glyph in sans-serif sp..." | Re-trigger Greptile

Wraps the ★ in its own .star-glyph span and switches that span to
a sans-serif font. The mono digit font was rendering ★ at 0.6rem
as a tiny smudge; sans-serif keeps the glyph crisp without bumping
its size relative to the digits.
@jdx jdx enabled auto-merge (squash) April 27, 2026 00:21
@jdx jdx merged commit d76df55 into main Apr 27, 2026
20 checks passed
@jdx jdx deleted the docs/star-glyph branch April 27, 2026 00:24
@jdx jdx mentioned this pull request Apr 27, 2026
jdx added a commit that referenced this pull request Apr 30, 2026
### 🐛 Bug Fixes

- **(hook)** do not stage fixes when fail_on_fix=true by
[@jdx](https://github.com/jdx) in
[#892](#892)
- use site domain for plausible data-domain by
[@jdx](https://github.com/jdx) in
[#886](#886)
- make text-mode progress output usable in CI by
[@jdx](https://github.com/jdx) in
[#890](#890)

### 📚 Documentation

- prefix GitHub star count with ★ glyph by
[@jdx](https://github.com/jdx) in
[#883](#883)

### 🔍 Other Changes

- **(release)** dedupe sponsor section in release notes by
[@jdx](https://github.com/jdx) in
[#881](#881)
- switch analytics from gtm/goatcounter to plausible by
[@jdx](https://github.com/jdx) in
[#885](#885)
- migrate to namespace.so runners by [@jdx](https://github.com/jdx) in
[#891](#891)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk release bookkeeping: version bumps, regenerated docs, and
lockfile dependency updates with no functional code changes in this PR.
> 
> **Overview**
> Bumps `hk` to **v1.44.3** and adds the corresponding `CHANGELOG.md`
release entry.
> 
> Regenerates versioned docs/CLI artifacts to reference `1.44.3`
(package URLs and generated `commands.json`/`index.md`) and updates
`Cargo.lock` with dependency resolution changes (notably `jni`,
`rustls*`, `reqwest`, `wasm-bindgen`, and `thiserror` unification).
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
ab7b72e. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: mise-en-dev <[email protected]>
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