Skip to content

Touch: move the ghost with a finger, place with a button or double tap - #60

Merged
malte9799 merged 1 commit into
mainfrom
claude/beautiful-banach-6c7253
Oct 14, 2026
Merged

malte9799 merged 1 commit into
mainfrom
claude/beautiful-banach-6c7253

Conversation

@malte9799

Copy link
Copy Markdown
Owner

Closes #59

What

On touch screens an item or blueprint in hand was dropped the instant the finger landed, and a drag stamped a trail. Now:

  • Touching or dragging only moves the ghost.
  • It is placed by the new Place button in the touch "in hand" bar, or by a double tap on the map.
  • Track (rail planner) keeps its drag-to-lay behaviour, and mouse input is unchanged.

renderer.placeHeld() is the new entry point; it falls back to the view centre if no touch has positioned the ghost yet.

Verification

tsc --noEmit for renderer and site, and the renderer tests, pass. Not tried on a real touch device.

Preview: https://malte9799.github.io/FactorioTools/preview/claude-beautiful-banach-6c7253/

Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: badc3ea1-c7d0-4ca7-86ac-1e21bc5deb0e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

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

Comment @coderabbitai help to get the list of available commands.

@malte9799
malte9799 merged commit e1ac68e into main Oct 14, 2026
5 checks passed
@malte9799
malte9799 deleted the claude/beautiful-banach-6c7253 branch October 14, 2026 19:05
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.

Touch screens: placing an item/blueprint should not happen instantly on finger-down

1 participant