GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix(short-input): hide selected text to prevent overlay collision by octo-patch · Pull Request #4318 · simstudioai/sim

Conversation

@octo-patch

@octo-patch octo-patch commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #3389

When users selected text in a ShortInput field and scrolled horizontally, the transparent input text became visible, creating a visual collision with the formatted overlay.

Root cause: ShortInput renders text as color: transparent to hide the raw input beneath a formatted overlay div. However, the browser's ::selection pseudo-element overrides color, so during text selection the transparent text becomes visible and "bleeds through" the overlay.

Fix: Add selection:text-transparent to the input's class list. This sets ::selection { color: transparent }, keeping the input text invisible even when selected, while the overlay continues to display the correctly formatted and highlighted text.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Changes

  • apps/sim/app/workspace/.../short-input/short-input.tsx: Added selection:text-transparent Tailwind class to the <Input> element.

Testing

  1. Add a block to the workflow (e.g. Webhook trigger)
  2. Open the panel and find any short-input field
  3. Type text longer than the field width
  4. Click and drag to select all the text, then scroll right
  5. Before fix: Raw unformatted text bleeds through the overlay
  6. After fix: Only the formatted overlay text is visible; no collision occurs

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
waleedlatif1 and others added 30 commits April 3, 2026 23:30
@waleedlatif1
v0.6.23: MCP fixes, remove local state in favor of server state, moth…
0b9019d
…ership workflow edits via sockets, ui improvements
@waleedlatif1
v0.6.24: copilot feedback wiring, captcha fixes
a54dcbe
@waleedlatif1
v0.6.25: cloudwatch, cloudformation, live kb sync, linear fixes, post…
28af223
…hog upgrade
@waleedlatif1
v0.6.26: ui improvements, multiple response blocks, docx previews, ol…
d889f32
…lama fix
@waleedlatif1
v0.6.27: new triggers, mothership improvements, files archive, queuei…
316bc8c
…ng improvements, posthog, secrets mutations
@waleedlatif1
v0.6.28: new docs, delete confirmation standardization, dagster integ…
3f508e4
…ration, signup method feature flags, SSO improvements
@TheodoreSpeaks
v0.6.29: login improvements, posthog telemetry (simstudioai#4026)
d6ec115
* feat(posthog): Add tracking on mothership abort (simstudioai#4023)

Co-authored-by: Theodore Li <[email protected]>

* fix(login): fix captcha headers for manual login  (simstudioai#4025)

* fix(signup): fix turnstile key loading

* fix(login): fix captcha header passing

* Catch user already exists, remove login form captcha
@waleedlatif1
v0.6.30: slack trigger enhancements, connectors performance improveme…
d7da35b
…nts, secrets performance, polling refactors, drag resources in mothership
@waleedlatif1
v0.6.31: elevenlabs voice, trigger.dev fixes, cloud whitelabeling for…
cf233bb
… enterprises
@waleedlatif1
v0.6.32: BYOK fixes, ui improvements, cloudwatch tools, jsm tools ext…
f8f3758
…ension
@waleedlatif1
v0.6.33: polling improvements, jsm forms tools, credentials reactquer…
3c8bb40
…y invalidation, HITL docs
@waleedlatif1
v0.6.34: trigger.dev fixes, CI speedup, atlassian error extractor
d33acf4
@waleedlatif1
v0.6.35: additional jira fields, HITL docs, logs cleanup efficiency
4f40c4c
@waleedlatif1
v0.6.36: new chunkers, sockets state machine, google sheets/drive/cal…
cbfab1c
…endar triggers, docs updates, integrations/models pages improvements
@waleedlatif1
v0.6.37: audit logs page, isolated-vm worker rotation, permission gro…
4309d06
…ups ui
@waleedlatif1
v0.6.38: models page
8b57476
@waleedlatif1
v0.6.39: billing fixes, tools audit, landing fix
e3d0e74
@waleedlatif1
v0.6.40: mothership tool loop, new skills, agiloft, STS, IAM integrat…
0ac0539
…ions, jira forms endpoints
@waleedlatif1
v0.6.41: webhooks fix, workers removal
3838b6e
@waleedlatif1
v0.6.42: mothership nested file reads, search modal improvements
fc07922
@waleedlatif1
v0.6.43: mothership billing idempotency, env var resolution fixes
3a1b1a8
@waleedlatif1
v0.6.44: streamdown, mothership intelligence, excel extension
46ffc49
@Sg312
v0.6.45: superagent, csp, brightdata integration, gemini response for…
010435c
…mat, logs performance improvements

fix(csp): add missing analytics domains, remove unsafe-eval, fix workspace CSP gap (simstudioai#4179)
fix(landing): return 404 for invalid dynamic route slugs (simstudioai#4182)
improvement(seo): optimize sitemaps, robots.txt, and core web vitals across sim and docs (simstudioai#4170)
fix(gemini): support structured output with tools on Gemini 3 models (simstudioai#4184)
feat(brightdata): add Bright Data integration with 8 tools (simstudioai#4183)
fix(mothership): fix superagent credentials (simstudioai#4185)
fix(logs): close sidebar when selected log disappears from filtered list; cleanup (simstudioai#4186)
@icecrasher321
Merge pull request simstudioai#4190 from simstudioai/staging
c0bc62c
v0.6.46: mothership streaming fixes, brightdata integration
@waleedlatif1
v0.6.46: mothership queueing, web vitals
387cc97
@waleedlatif1
v0.6.47: files focusing, documentation, opus 4.7
2dbc7fd
@waleedlatif1
v0.6.48: import csv into tables, subflow fixes, CSP updates
8a50f18
@waleedlatif1
v0.6.49: deploy sockets event, resolver, logs improvements, monday.co…
dcf3302
…m integration, atlassian triggers
@icecrasher321
v0.6.50: ppt/doc/pdf worker isolation, docs, chat, sidebar improvements
bc09865
@waleedlatif1
v0.6.51: tables improvements, billing fixes, 404 pages, code hygiene
5f56e46
waleedlatif1 and others added 12 commits April 21, 2026 20:25
@waleedlatif1
v0.6.52: data retention, docs updates, slack manifest generator, secu…
ca3bbf1
…rity hardening, contact page, 404 page, access control, SES, SNS
@waleedlatif1
v0.6.53: permissions groups migration, docs updates
bbf400f
@icecrasher321
Merge pull request simstudioai#4261 from simstudioai/staging
7c619e7
v0.6.54: migration error logs
@icecrasher321
v0.6.54: mothership tracing, db pool size increase
64cfda5
@waleedlatif1
v0.6.55: standardize monorepo conventions, api key hash, thinking tex…
7ca736a
…t for mothership
@waleedlatif1
v0.6.56: data retention improvements, tables column double click resi…
6066fc1
…ze, subagent thinking, files sorting, agentphone integration
@waleedlatif1
Merge pull request simstudioai#4285 from simstudioai/staging
3422f64
fix(db): revert statement_timeout startup options breaking pooled connections (simstudioai#4284)
@TheodoreSpeaks
Merge pull request simstudioai#4293 from simstudioai/staging
595c4c3
v0.6.57: mothership reliability, ashby refactor, tables row count, copilot id fix, bun upgrade
@icecrasher321
v0.6.58: queue abort state machine improvement, contributing guide
d6c1bc2
@icecrasher321
v0.6.59: gpt 5.5, security hardening, parallel subagents rendering
58a3ae2
@icecrasher321
v0.6.60: copilot security improvements, slack canvas ops, retention j…
489f2d3
…obs fixes
octo-patch @octo-patch
fix(short-input): hide selected text to prevent overlay collision
9c8e51e
The ShortInput component uses a transparent input layer beneath a
formatted overlay. When users selected text and scrolled horizontally,
the browser's ::selection pseudo-element overrode `color: transparent`,
making raw unformatted text visible and causing visual collision with
the overlay.

Adding `selection:text-transparent` keeps the input text invisible
under selection, while the overlay continues to display the correctly
formatted highlighted text.

Fixes simstudioai#3389

Co-Authored-By: Octopus <[email protected]>
@vercel

vercel Bot commented Apr 28, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Apr 28, 2026 3:49am

Request Review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vercel
vercel Bot temporarily deployed to Preview April 28, 2026 03:49 Inactive
@cursor

cursor Bot commented Apr 28, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

PR Summary

Low Risk
Low risk: a single Tailwind class change to ShortInput styling to prevent selected text from showing through the formatted overlay; no logic or data flow changes.

Overview
Fixes a visual bug in ShortInput where selected text could become visible and overlap the formatted overlay during horizontal scrolling.

This updates the input styling to also make ::selection text transparent via selection:text-transparent, keeping the raw input consistently hidden.

Reviewed by Cursor Bugbot for commit 9c8e51e. Bugbot is set up for automated code reviews on this repo. Configure here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@greptile-apps

greptile-apps Bot commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes a visual bug in ShortInput where selecting text caused the raw (transparent) input characters to become visible and overlay the formatted display div, because the browser's ::selection pseudo-element overrides color. The one-line fix adds selection:text-transparent to keep the input text invisible during selection. The change is correct, minimal, and well-scoped to the component's existing pattern of using text-transparent plus an absolute overlay.

Confidence Score: 5/5

Safe to merge — single-line, non-breaking CSS fix with no logic changes.

The change is a one-line Tailwind class addition that directly addresses the described bug without touching any logic, state, or APIs. The fix correctly targets the ::selection pseudo-element and is consistent with the existing text-transparent pattern already in the component. No regressions are plausible.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/short-input/short-input.tsx Adds selection:text-transparent Tailwind class to the hidden input so browser ::selection no longer overrides color: transparent, preventing raw text from bleeding through the formatted overlay on horizontal scroll.

Sequence Diagram

sequenceDiagram
    participant U as User
    participant I as Input (text-transparent)
    participant O as Overlay div (formatted text)

    U->>I: Types text
    I->>O: Sync scroll position (handleScroll)
    O->>U: Renders formatted/highlighted text

    Note over U,I: User selects text + scrolls
    I-->>U: Before fix — ::selection overrides color, raw text bleeds through overlay
    I-->>U: After fix — selection:text-transparent keeps color transparent during selection
Loading

Reviews (1): Last reviewed commit: "fix(short-input): hide selected text to ..." | Re-trigger Greptile

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@icecrasher321
icecrasher321 changed the base branch from main to staging April 28, 2026 03:53
@gitguardian

gitguardian Bot commented Apr 28, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

Since your pull request originates from a forked repository, GitGuardian is not able to associate the secrets uncovered with secret incidents on your GitGuardian dashboard.
Skipping this check run and merging your pull request will create secret incidents on your GitGuardian dashboard.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
29606901 Triggered Generic High Entropy Secret a54dcbe apps/sim/providers/utils.test.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider

  • following these best practices for managing and storing secrets including API keys and other credentials
  • install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.

🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1

waleedlatif1 commented Apr 28, 2026

Copy link
Copy Markdown
Collaborator

@octo-patch please rebase against origin staging. Thanks for the contribution!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1
waleedlatif1 merged commit 3afcad2 into simstudioai:staging Apr 28, 2026
13 of 14 checks passed

Uh oh!

There was an error while loading. Please reload this page.

This was referenced Apr 29, 2026
v0.6.61: SAP integration, live URLs for browser use, 5xx error categorizations, mothership positional table row insertion, CI improvements, org-external users, file viewer improvements #4330
Merged
feat(firecrawl): add parse operation and revert short-input selection style #4340
Merged
TheodoreSpeaks added a commit that referenced this pull request Apr 30, 2026
@TheodoreSpeaks @claude
Merge origin/staging into feat/remove-search-context-menu
97844db
Resolved conflicts:
- user-input.tsx: kept HEAD's mention-menu logic (syncMentionState, mentionRangeRef, useCallback handleInputChange) over staging's simplified inline handleInputChange
- short-input.tsx: kept HEAD's selection:text-transparent fix from #4318

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@waleedlatif1 waleedlatif1 mentioned this pull request May 2, 2026
revert(short-input): remove selection:text-transparent (#4318) #4398
Merged
5 of 6 tasks
waleedlatif1 added a commit that referenced this pull request May 2, 2026
@waleedlatif1
revert(short-input): remove selection:text-transparent (#4318) (#4398)
9eb55e2

This branch was previously deployed

1 inactive deployment
Preview — 9c8e51e2 Deployed Apr 28, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] Short Input Component Text Distortion on selection

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@octo-patch @waleedlatif1 @TheodoreSpeaks @Sg312 @icecrasher321
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.