Skip to content

Fix: TryIt Modal server filtering + Navbar active state#2824

Open
srivastavaayush1611LT wants to merge 2 commits intoLambdaTest:stagefrom
srivastavaayush1611LT:fix/tryit-modal-server-dropdown
Open

Fix: TryIt Modal server filtering + Navbar active state#2824
srivastavaayush1611LT wants to merge 2 commits intoLambdaTest:stagefrom
srivastavaayush1611LT:fix/tryit-modal-server-dropdown

Conversation

@srivastavaayush1611LT
Copy link
Copy Markdown

@srivastavaayush1611LT srivastavaayush1611LT commented Apr 16, 2026

Summary

  1. TryIt Modal: Add v1/v2 server filtering (matching EndpointDetail behavior)
  2. Navbar: Fix active link highlighting not working on production

Changes

TryIt Modal

  • Add v1/v2 endpoint detection logic
  • Filter servers dropdown to show only matching API version
  • Server dropdown appears when multiple servers available

Navbar

  • Add !important to navLinkActive styles for CSS specificity
  • Make isActiveLink function more robust with null checks and case normalization

Test plan

  • TryIt Modal: V1 endpoints show only v1 server URLs
  • TryIt Modal: V2 endpoints show only v2 server URLs
  • Navbar: API Reference link highlights orange when on API doc pages
  • Navbar: Other nav links highlight correctly on their respective pages

🤖 Generated with Claude Code

- Add v1/v2 endpoint detection logic
- Filter servers to show only matching API version
- Add server dropdown when multiple servers available
- Update curl generation and send request to use selected server

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add !important to navLinkActive styles for CSS specificity
- Make isActiveLink function more robust with null checks and case normalization

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@srivastavaayush1611LT srivastavaayush1611LT changed the title Fix: Add server version filtering to TryIt Modal Fix: TryIt Modal server filtering + Navbar active state Apr 16, 2026
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.

1 participant