Skip to content

fix: keep WebView2 visible when auto-hide flyouts are collapsed - #341

Merged
martinsafsten-codescene merged 1 commit into
mainfrom
fix/airspace-guard-collapsed-autohide
Oct 28, 2026
Merged

martinsafsten-codescene merged 1 commit into
mainfrom
fix/airspace-guard-collapsed-autohide

Conversation

@martinsafsten-codescene

Copy link
Copy Markdown
Contributor

Summary

  • The airspace guard from fix: keep WebView2 from covering auto-hidden VS windows #322 hid the WebView2 whenever any auto-hidden tool window overlapped a CodeScene pane. DTE reports collapsed auto-hide windows as visible, with their flyout rectangle, so a single collapsed window could hide the webview permanently. The CodeScene tool window then rendered blank across restarts and reinstalls, since the layout lives in VS settings.
  • An auto-hide window now only counts as an overlay while its IVsWindowFrame reports being on screen. If the on-screen state can't be read, the webview stays visible.
  • The guard now logs a debug line naming the overlapping window when it hides the webview, and another when it shows it again.
  • Follow-up to CS-11278. Customer-reported against 0.9.0, and present since 0.8.3.

Test plan

  • make iter passes; Code Health for the guard stays at 10
  • Repro: CodeScene docked on the right, another tool window auto-hidden on the right edge, flyout opened then collapsed. Before: the pane stays blank. After: the pane comes back once the flyout collapses
  • CS-11278 still fixed: CodeScene as a document tab, open an auto-hide flyout, and the flyout is not covered

The airspace guard treated every auto-hidden tool window as an overlay, because DTE reports collapsed auto-hide windows as visible with their flyout rectangle. Any collapsed window whose flyout overlapped a CodeScene pane hid the webview permanently, leaving the tool window blank.

Only count auto-hide windows whose frame is on screen, and log when the guard hides or shows the webview.
@martinsafsten-codescene
martinsafsten-codescene merged commit 198b077 into main Oct 28, 2026
3 checks passed
@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@codescene-access codescene-access Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No application code in the PR — skipped Code Health checks.

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

@codescene-access codescene-access Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Coverage Gates Passed
New & Changed Code Coverage: 100% (required = 95%)

Review details by gate
  • New & Changed Code Coverage
    • Pass/Fail Reason:
      No executable lines of code changed. (The change touched 0 files).

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