Skip to content

fix highlight right sidebar#1092

Merged
dgarcia360 merged 1 commit intoscylladb:masterfrom
dgarcia360:fix-right-navigation
May 7, 2024
Merged

fix highlight right sidebar#1092
dgarcia360 merged 1 commit intoscylladb:masterfrom
dgarcia360:fix-right-navigation

Conversation

@dgarcia360
Copy link
Copy Markdown
Collaborator

Motivation

Since 1.7.1, the right-side navigation highlighting was not behaving as expected. When a heading is clicked, sometimes it incorrectly highlights the previous section instead of the current one.

Example: When clicking https://sphinx-theme.scylladb.com/stable/deployment/production.html#setting-up-a-custom-domain, "Installation" is highlighted instead of "Setting up a custom domain".

How to test

  1. Clone the PR.
  2. Build the docs.
  3. Open http://127.0.0.1:5500/deployment/production/#setting-up-a-custom-domain
  4. "Setting up a custom domain" should be highlighted, not "Installation".

Copy link
Copy Markdown
Collaborator

@annastuchlik annastuchlik left a comment

Choose a reason for hiding this comment

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

LGTM

@dgarcia360 dgarcia360 merged commit 1c80df1 into scylladb:master May 7, 2024
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