Skip to content

Conversation

@ethanpalm
Copy link
Collaborator

@ethanpalm ethanpalm commented Dec 23, 2025

Documentation changes

Elf mode


For Reviewers

When reviewing documentation PRs, please consider:

✅ Technical accuracy

  • Code examples work as written
  • Commands and configurations are correct
  • Links resolve to the right destinations
  • Prerequisites and requirements are accurate

✅ Clarity and completeness

  • Instructions are clear and easy to follow
  • Steps are in logical order
  • Nothing important is missing
  • Examples help illustrate the concepts

✅ User experience

  • A new user could follow these docs successfully
  • Common gotchas or edge cases are addressed
  • Error messages or troubleshooting guidance is helpful

Note

Updates branding for the docs site to a holiday theme.

  • Points logo.light and logo.dark in docs.json to logo/holiday-light.svg and logo/holiday-dark.svg
  • Adds new SVG assets: logo/holiday-light.svg, logo/holiday-dark.svg

Written by Cursor Bugbot for commit 64dae79. This will update automatically on new commits. Configure here.

@@ -0,0 +1,34 @@
<svg width="1747" height="438" viewBox="0 0 1747 438" fill="none" xmlns="http://www.w3.org/2000/svg">
Copy link

Choose a reason for hiding this comment

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

Holiday logos have mismatched dimensions between themes

The holiday logo SVGs have inconsistent dimensions. holiday-dark.svg has width="1748" height="435" while holiday-light.svg has width="1747" height="438". This 1-pixel width and 3-pixel height difference can cause visual shifting when users toggle between light and dark modes. The original logos maintained identical dimensions (1682x368) across both themes.

Additional Locations (1)

Fix in Cursor Fix in Web

@ethanpalm ethanpalm merged commit 8084ac2 into main Dec 23, 2025
5 checks passed
@ethanpalm ethanpalm deleted the holiday-logo branch December 23, 2025 22:41
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.

2 participants