fix(docs): Resolve logo overlap on tablet view#5853
Conversation
Signed-off-by: Dedy F. Setyawan <dedyfajars@gmail.com>
✅ Deploy Preview for localai ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Good catch! however I think we want to still override the logo from the theme, maybe we can try to apply a different class to make the svg display in a smaller view? |
Signed-off-by: Dedy F. Setyawan <dedyfajars@gmail.com>
|
Thanks for the feedback! I've updated the PR to use a dedicated CSS file ( Please let me know if you have any further feedback. |
| {{ end }} | ||
| </div> | ||
| <div class="big"> | ||
| <div class="big header-logo"> |
There was a problem hiding this comment.
to be fair I would avoid to bring in the template header override just for a single class definition - can we maybe define it in the top-header.html file, or inline as it's a one-off?
There was a problem hiding this comment.
Got it! Makes sense to avoid the full partial override for this.
I've updated the PR to use a <style> block directly in top-header.html.
Let me know if that works!
Signed-off-by: Dedy F. Setyawan <dedyfajars@gmail.com>
Description
This PR addresses a display bug where the
logo.svgwas improperly scaled and caused content overlap on tablet devices.The problem manifested when viewing the documentation on tablet screen sizes, where the original
logo.svgrendered too large and overlapped portions of the main content.This commit resolves the issue by replacing the oversized
logo.svgwithmark.svgand explicitly setting its width to22px. This ensures the logo is appropriately sized for tablet displays, effectively eliminating the overlap issue and achieving visual consistency with the mobile version of the site.This PR fixes # N/A
Notes for Reviewers
To verify this fix:
mark.svg(notlogo.svg), appears correctly sized, and no longer overlaps the content. Also, observe if its appearance is consistent with the mobile view.Screenshots:

Before:
After:

Signed commits