Skip to content

Fix/nav collapse button visibility#357

Merged
JakeSCahill merged 3 commits intomainfrom
fix/nav-collapse-button-visibility
Jan 27, 2026
Merged

Fix/nav collapse button visibility#357
JakeSCahill merged 3 commits intomainfrom
fix/nav-collapse-button-visibility

Conversation

@JakeSCahill
Copy link
Copy Markdown
Contributor

@JakeSCahill JakeSCahill commented Jan 26, 2026

This pull request makes minor improvements to the Bloblang Playground UI, focusing on consistency in dropdown placeholder text and a small layout adjustment for the navigation panel.

  • UI/UX consistency:

    • Changed the placeholder text in the sample dropdown to "Choose an example..." for consistency across the UI.
    • Removed the default <option value="">Choose an example...</option> from the dropdown since the placeholder now serves this purpose.

  • Layout adjustment:

    • Updated the navigation panel's height calculation in nav.css to account for the announcement bar, ensuring proper layout when the announcement bar is present.

JakeSCahill and others added 2 commits January 25, 2026 11:33
@JakeSCahill @claude
Fix nav collapse button visibility when announcement bar is present
1ecb210 
The nav-container height was not accounting for the announcement bar height,
causing the container to extend beyond the viewport and hiding the collapse
button at the bottom. Updated the height calculation to subtract the
announcement bar height on desktop viewports.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@JakeSCahill @claude
Remove placeholder option from Bloblang playground dropdown
12e0544 
The "Choose an example..." option was appearing in the dropdown list
because it was defined as an HTML option. Removed the HTML placeholder
option and updated Choices.js configuration to use "Choose an example..."
as the placeholderValue. This ensures the placeholder text displays
correctly but doesn't appear in the dropdown list itself.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@netlify
Copy link
Copy Markdown

netlify bot commented Jan 26, 2026 •
edited
Loading

✅ Deploy Preview for docs-ui ready!

Name Link
🔨 Latest commit 6480920
🔍 Latest deploy log https://app.netlify.com/projects/docs-ui/deploys/6978d2de98e9b60008e1893f
😎 Deploy Preview https://deploy-preview-357--docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 1 paths audited
Performance: 28 (no change from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (🟢 up 17 from production)
SEO: 89 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@JakeSCahill JakeSCahill requested review from a team and paulohtb6 January 26, 2026 08:53
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 26, 2026 •
edited
Loading

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
📝 Walkthrough

Walkthrough

The pull request makes two independent UI adjustments. First, the desktop navigation container's height and top positioning calculations in the CSS are updated to account for the announcement bar height using CSS variables. Second, the Bloblang playground example dropdown is refactored by removing an empty option element and updating the Choices.js placeholder text from "Select an example" to "Choose an example...".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • paulohtb6
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fix/nav collapse button visibility' is related to the main change (navigation panel height adjustment) but does not capture the full scope of the PR which also includes UI/UX consistency improvements to the Bloblang Playground dropdown.
Description check ✅ Passed The description clearly explains the changeset including UI/UX consistency improvements and layout adjustments, with specific details about what was changed and why.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@JakeSCahill JakeSCahill merged commit bfaed69 into main Jan 27, 2026
6 checks passed
@JakeSCahill JakeSCahill deleted the fix/nav-collapse-button-visibility branch January 27, 2026 15:31
@coderabbitai coderabbitai bot mentioned this pull request Feb 3, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@paulohtb6 paulohtb6 paulohtb6 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JakeSCahill @paulohtb6