Skip to content

docs: prefix GitHub star count with ★ glyph#9417

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

docs: prefix GitHub star count with ★ glyph#9417
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 "★ 27.2k" instead of just "27.2k".

While here, switches the star badge from innerHTML to textContent — no behavior change for a plain-string value, slightly safer.


Note

Low Risk
Low risk, docs-theme-only UI tweak that just adjusts rendered text/CSS for the GitHub star badge and slightly hardens DOM insertion against HTML injection.

Overview
Updates the docs nav GitHub social link star badge to render as ★ <count> by inserting a dedicated .star-glyph span before the star count.

Adds styling for the glyph in custom.css and switches the badge population away from innerHTML to DOM nodes/text content.

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

Adds a small ★ (U+2605) before the GitHub stargazer count in the
top-nav social link, e.g. "★ 27.2k" instead of just "27.2k".
Switches the star badge to textContent (no behavior change for the
plain-string value, slightly safer than innerHTML).
@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
Contributor

greptile-apps Bot commented Apr 26, 2026

Greptile Summary

This PR adds a ★ glyph before the GitHub star count in the docs top-nav, and replaces the innerHTML assignment with a safer DOM construction using append() with a text node.

Confidence Score: 5/5

Safe to merge — purely a docs UI cosmetic change with no business logic impact.

No P0 or P1 issues found. The change is small, well-scoped, and the switch from innerHTML to append() with a plain string is strictly safer.

No files require special attention.

Important Files Changed

Filename Overview
docs/.vitepress/theme/index.ts Replaces innerHTML with safer DOM construction; creates a star-glyph span (aria-hidden) and appends it alongside the star count string using Element.append().
docs/.vitepress/theme/custom.css Adds scoped CSS for the new .star-glyph span — sets font-family for glyph rendering and a small margin-right for spacing.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Fetch starsData] --> B{githubLink exists\n& no .star-count yet?}
    B -- No --> Z[Skip]
    B -- Yes --> C[Create span.star-count]
    C --> D[Create span.star-glyph\ntextContent = ★\naria-hidden = true]
    D --> E[append glyph + starsData.stars\nas text node]
    E --> F[Append .star-count\nto githubLink]
Loading

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

Hyperfine Performance

mise x -- echo

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.23 x -- echo 23.8 ± 0.5 23.0 30.1 1.00
mise x -- echo 24.5 ± 0.4 23.8 28.1 1.03 ± 0.03

mise env

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.23 env 23.3 ± 0.5 22.5 27.2 1.00
mise env 24.0 ± 0.4 23.2 25.7 1.03 ± 0.03

mise hook-env

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.23 hook-env 24.0 ± 0.4 23.3 27.8 1.00
mise hook-env 24.9 ± 0.5 23.9 30.0 1.04 ± 0.03

mise ls

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.23 ls 24.3 ± 0.4 23.5 25.9 1.00
mise ls 25.7 ± 0.4 24.7 30.3 1.06 ± 0.02

xtasks/test/perf

Command mise-2026.4.23 mise Variance
install (cached) 159ms 164ms -3%
ls (cached) 82ms 84ms -2%
bin-paths (cached) 84ms 86ms -2%
task-ls (cached) 826ms 822ms +0%

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 46e4f07 into main Apr 27, 2026
37 checks passed
@jdx jdx deleted the docs/star-glyph branch April 27, 2026 00:31
mise-en-dev added a commit that referenced this pull request Apr 27, 2026
### 🚀 Features

- **(ls-remote)** add `prereleases` setting and `--prerelease` flag by
@jdx in [#9415](#9415)

### 🐛 Bug Fixes

- **(http)** retry transient HTTP failures with backoff and warn on
rescue by @jdx in [#9414](#9414)
- **(release)** purge mise.en.dev CDN zone after each S3 publish by @jdx
in [#9416](#9416)

### 📚 Documentation

- prefix GitHub star count with ★ glyph by @jdx in
[#9417](#9417)
- update intro messaging by @jdx in
[#9418](#9418)
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