Skip to content

fix(docs-ui): move nav links to sidebar and fix mobile responsiveness#433

Open
tharropoulos wants to merge 9 commits intotypesense:masterfrom
tharropoulos:navbar-fixes
Open

fix(docs-ui): move nav links to sidebar and fix mobile responsiveness#433
tharropoulos wants to merge 9 commits intotypesense:masterfrom
tharropoulos:navbar-fixes

Conversation

@tharropoulos
Copy link
Collaborator

Change Summary

  • Improve search box behavior and spacing, especially on mobile and narrow screens
  • Hide the search shortcut indicator on smaller screens unless the input is focused
  • Improve navbar layout and alignment to match the sub-link spacing
  • Add mobile-specific logo sizing support
  • Polish version dropdown styling and placement in the sidebar
  • Minor style cleanups and consistency fixes across VuePress components

Screenshots:

Mobile search UI

image

Mobile normal UI

image

Full-size search UI

image

PR Checklist

Copy link
Member

@jasonbosco jasonbosco left a comment

Choose a reason for hiding this comment

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

@tharropoulos Could you fix the margins / padding here:

Image

I also noticed that some pages on mobile (not all) are missing the search bar:

Image

Could you also fix the conflict

@tharropoulos
Copy link
Collaborator Author

@jasonbosco This had to do with how codeblocks and table tabs were overflowing, width wasn't enforced, leading to the whole page being horizontally scrollable, and since the navbar was a flexbox, the elements of the navbar were pushed to the end of the page. This was already the case for pages with large tables and codeblocks (try checking the /collections page from a phone currently).

Commits 3b46f81, c4a3d1f and 4770dd8 address this, along with some other general UI improvements for responsiveness.

@tharropoulos
Copy link
Collaborator Author

tharropoulos commented Feb 2, 2026

@tharropoulos Could you fix the margins / padding here

By fixing, do you mean increasing? 😅 Horizontal padding should be equal to that of the page's title IMO (one general padding and an inner because of the pink background). In terms of vertical?

@jasonbosco
Copy link
Member

Mainly the padding inside the blue box, and the margin after the heading

- refine padding and margins for titles, content, and subcategories to
  match
@tharropoulos
Copy link
Collaborator Author

@jasonbosco updated the spacing to match on c690290
image

@jasonbosco
Copy link
Member

@tharropoulos I just tried running the latest code locally, and I still see the overflow issue on the collections page

@tharropoulos
Copy link
Collaborator Author

@jasonbosco What were the viewport's dimensions?

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