Skip to content

Stop blanking out the main browser (BL-16047)#7791

Open
andrew-polk wants to merge 1 commit intomasterfrom
BL16047_TCblank
Open

Stop blanking out the main browser (BL-16047)#7791
andrew-polk wants to merge 1 commit intomasterfrom
BL16047_TCblank

Conversation

@andrew-polk
Copy link
Copy Markdown
Contributor

@andrew-polk andrew-polk commented Mar 30, 2026


Open with Devin

This change is Reviewable

@andrew-polk
Copy link
Copy Markdown
Contributor Author

src/BloomExe/web/controllers/AppApi.cs line 244 at r1 (raw file):

                return;
            }

The changes in this file are directly related to the fix, but they are things which the AI found which should have been in place while we were analyzing the real problem.

@andrew-polk
Copy link
Copy Markdown
Contributor Author

src/BloomExe/Edit/EditingView.cs line 1605 at r1 (raw file):

            _mainBrowser?.RunJavascriptFireAndForget(
                "workspaceBundle.switchContentPage('about:blank');"
            );

The original code here just blanked out the edit view's browser. It was mistakenly applied to the whole main browser during the single-browser conversion.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

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