Skip to content

Conversation

schopr9
Copy link

@schopr9 schopr9 commented Oct 1, 2025

Resolves #9118

Description

Fixes the statusbar top border visibility issue when toggling the organization sidebar.

Problem: When the organization sidebar is toggled, the statusbar's top border color disappears due to CSS Grid layout changes affecting the divide-x utility behavior.

Solution: Added conditional border-t styling to the statusbar

Changes Made

  • Modified packages/insomnia/src/routes/organization.tsx to conditionally apply border-solid border-t border-[--hl-md] classes to the statusbar
  • Used existing isOrganizationSidebarOpen state to control border visibility

Testing

  • ✅ Linting passes
  • ✅ TypeScript compilation passes
  • ✅ Manual testing shows border appears/disappears correctly when toggling sidebar

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Screenshots/GIFs

Screen.Recording.2025-10-01.at.6.09.40.PM.mov

-->

@CLAassistant
Copy link

CLAassistant commented Oct 1, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

sentry-io bot commented Oct 1, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: packages/insomnia/src/routes/organization.tsx

Function Unhandled Issue
Component ReferenceError: InsomniaUpdateTabProvider is not defined /organization/org_scratchpad/project/proj_scratchpad/workspace/wrk_scratchp...
Event Count: 3 Affected Users: 0

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.

break layout when both turn toggle off

2 participants