Skip to content

fix(desktop-gpui): mirror the teleprompter script when Mirror text is on - #4250

Open
richiemcilroy wants to merge 4 commits into
mainfrom
fix/gpui-teleprompter-mirror
Open

richiemcilroy wants to merge 4 commits into
mainfrom
fix/gpui-teleprompter-mirror

Conversation

@richiemcilroy

@richiemcilroy richiemcilroy commented Nov 14, 2026 •

Copy link
Copy Markdown
Member

The teleprompter's Mirror text toggle saved but changed nothing. This adds a GPUI patch for a horizontal mirror and uses it on the script, with clicks and the IME box following the mirrored text. Draft while proof runs; full write-up to follow.

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge; no new actionable issue was found.

Summary

Makes the teleprompter’s Mirror text toggle flip the script.

  • The teleprompter script now mirrors while the Mirror setting is on.

The earlier test-coverage thread was manually resolved without explanation.

Reviews (2) · Last reviewed commit: "ci: run the GPUI mirror patch tests" · Reviewed by Greptile

@sandchest-com

sandchest-com Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

Sandchest

Sandchest reviewed 36b5775: nothing exploitable found in this change.

Sandchest reviews every change for vulnerabilities an attacker could use · Review details

Comment thread apps/desktop-gpui/src/ui/text_input.rs Resolved

@capy-ai capy-ai 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.

Capy found no issues.

View 1 other finding in Capy.

Open in Capy

@vercel

vercel Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
cap-status Skipped Skipped Nov 14, 2026 6:51am UTC

@capy-ai capy-ai 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.

Capy found no issues.

View 2 other findings in Capy.

Open in Capy

@richiemcilroy

Copy link
Copy Markdown
Member Author

hey @greptileai, please re-review the PR

@vercel
vercel Bot temporarily deployed to Preview – cap-status November 14, 2026 06:51 Inactive

@capy-ai capy-ai 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.

Capy found no issues.

Open in Capy

This branch was successfully deployed

1 active (outdated) and 1 inactive deployments
Preview – cap-status — 36b57756 Deployed Nov 14, 2026 by vercel[bot]
Preview – cap-web — f306bd41 Deployed Nov 14, 2026 by vercel[bot]
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