Skip to content

🛡️ Sentinel / 🎨 Palette: Consolidated QTextEdit Plain Text Batch - #227

Closed
dieterolson wants to merge 3 commits into
mainfrom
consolidated/qtextedit-html-injection-batch
Closed

🛡️ Sentinel / 🎨 Palette: Consolidated QTextEdit Plain Text Batch#227
dieterolson wants to merge 3 commits into
mainfrom
consolidated/qtextedit-html-injection-batch

Conversation

@dieterolson

Copy link
Copy Markdown
Contributor

Consolidating open QTextEdit plain text & HTML injection PRs into a single batch to reduce CI/CD load.

Closes #220, Closes #221, Closes #222, Closes #223, Closes #224, Closes #225

google-labs-jules Bot and others added 3 commits August 5, 2026 06:01
Added `setAcceptRichText(False)` to `QTextEdit` widgets across the application (chat panel, setup logs, hub dialog logs, and settings prompt). By default, `QTextEdit` accepts HTML (via drag/drop or copy/paste), which could be leveraged to inject unexpected rich text UI elements or load external images if not strictly handling plain text.

Co-authored-by: dieterolson <198168927+dieterolson@users.noreply.github.com>
Added `setAcceptRichText(False)` to `QTextEdit` widgets across the application (chat panel, setup logs, hub dialog logs, and settings prompt). By default, `QTextEdit` accepts HTML (via drag/drop or copy/paste), which could be leveraged to inject unexpected rich text UI elements or load external images if not strictly handling plain text.

Co-authored-by: dieterolson <198168927+dieterolson@users.noreply.github.com>
@dieterolson

Copy link
Copy Markdown
Contributor Author

Incorporated into consolidated PR #229.

auto-merge was automatically disabled August 13, 2026 03:26

Pull request was closed

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