fix: replace quipu logo and center text#3
Conversation
| font-size: 11px; | ||
| font-weight: 600; | ||
| color: var(--scalar-sidebar-color-2); | ||
| padding-top: 3px; |
There was a problem hiding this comment.
This is for centering vertically the text at the right side of the logo
| gap: 10px; | ||
| } | ||
| .quipu-sidebar-logo img { height: 24px; } | ||
| .quipu-sidebar-logo img { height: 30px; } |
There was a problem hiding this comment.
The new logo looks smaller than the previous one, so I have increased a bit the size
There was a problem hiding this comment.
Pull request overview
Updates the public API docs landing page styling to accommodate the new Quipu logo and visually center the “API v1” label next to it in the sidebar header.
Changes:
- Increased the sidebar logo image height from 24px to 30px.
- Added a small top padding to the “API v1” label to improve vertical alignment with the updated logo.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
talken
left a comment
talken
left a comment
There was a problem hiding this comment.
APPROVE ✅
Purely cosmetic branding change in the Scalar-based API docs renderer: swaps the binary logo.png and adjusts two CSS rules in index.html (logo height 24px→30px at line 67; padding-top: 3px on .api-label at line 73 to align with the taller logo). No OpenAPI spec, endpoint definitions, parameters, types, examples, or the application/vnd.quipu.v1+json accept header are touched — no API-contract accuracy risk. The changed selectors (.quipu-sidebar-logo img, .quipu-sidebar-logo .api-label) match the JS-injected elements (index.html:124-128) and the existing .dark-mode invert filter still applies.
Non-blocking:
- The height bump + fixed
padding-top: 3pxrely on the new PNG's intrinsic aspect ratio matching the old one; worth a quick visual confirm in light + dark mode (the PR screenshots cover this).
Description
Update the logo of the public API
Before:
After:
Motivation and Context