feat: add pinch-to-zoom support to editor by gahanad · Pull Request #2963 · Acode-Foundation/Acode

gahanad · 2026-10-09T08:27:45Z

Description

Adds pinch-to-zoom support to the editor to make editing more convenient on mobile devices.

Users can adjust the editor's font size using a two-finger pinch gesture, improving readability and usability on smaller screens.

Changes Made

Added a pinch-to-zoom extension for the editor.
Dynamically adjusts the editor font size based on the pinch gesture.
Enforces minimum and maximum font-size limits.
Saves the updated font size setting when the gesture ends.
Cleans up touch event listeners when the extension is destroyed.
Added unit tests covering pinch-zoom behavior and font-size calculations.

Testing

Added unit tests for pinch-zoom functionality.
Verified the targeted unit tests pass.
Manually tested pinch-to-zoom on an Android device.
Verified the full test suite and CI checks.

Related Issue

Closes #2426

greptile-apps · 2026-10-09T08:32:44Z

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; both numbered previous findings are fixed.

Summary

Adds two-finger pinch zoom to the editor, with bounded font sizes, live previews, and saving when the gesture ends.

You can pinch the editor text to change its font size.
Two-finger gestures cancel pending text-selection actions.

_{Reviews (6) · Last reviewed commit: "Fixed corrections for pinchZoom" · Reviewed by Greptile}

bajrangCoder · 2026-10-11T11:11:54Z

There several accidental zoom in/zoom out

screen-20261011-164227.mp4

gahanad · 2026-10-13T21:28:37Z

@greptile

bajrangCoder · 2026-10-15T15:47:13Z

+		persistFontSize();
+	}
+
+	const target = view.contentDOM;


Touch events keep targeting the element where each finger first landed, even after that element is removed from the DOM. Since move/end/cancel are only listened for on contentDOM, the gesture can get stuck:

Two fingers start on lines in the lower part of the editor; spread to ~2x.

The preview font change + requestMeasure() makes CM recompute a smaller viewport (visible area + ~500px margin), and the .cm-line under a finger is unmounted.

That finger's touchmove/touchend now dispatch to a detached node and never reach contentDOM.

Result: gesture.pinching stays true, the inline preview is never cleared or persisted, and every subsequent two-finger touchstart bails on if (gesture.pinching ...) — pinch is dead until the editor state is swapped.

Suggest attaching the move/end/cancel listeners to window/document (capture) when a pinch starts and removing them in endPinch/destroy.

bajrangCoder · 2026-10-15T16:04:15Z

Also isn't the line number area and editor area pinch-to-zoom at same time for better and smooth feel? as currently line number area adjusted itself after leaving zoom.

Also in some cases some accidental scrolls maybe using three finger/two finger etc does nothing but normally in acode those just scrolls so removing that will be a bit weird(if there is no valid reason)

…on#2961) Bumps ghcr.io/devcontainers/features/node from 2.1.0 to 2.1.1. --- updated-dependencies: - dependency-name: ghcr.io/devcontainers/features/node dependency-version: 2.1.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Updated Discord badge URL to a new endpoint.

…e-Foundation#2966) * refactor(browser): keep a list of devices in the emulator device list * refactor(browser): register each device when added to the list * feat(browser): apply the saved default preview size once layout is ready * feat(browser): save and restore a default device preview size * feat(browser): add "Set as Default" and "Clear Default" menu items * feat(browser): handle "Set as Default" and "Clear Default" actions * feat(browser): show default menu items only while Devices mode is on * feat(browser): open the preview in Devices mode when a default is set * feat(browser): restore the default preview mode on startup * refactor(browser): allow applying desktop mode without reloading * refactor(browser): make the reload in setDesktopMode optional * fix(browser): apply saved desktop mode before the first load to avoid a reload * fix(browser): only apply dimensions after layout, not desktop mode * fix(browser): re-fit saved presets to the current screen on restore * fix(browser): reload once after the saved size is applied * fix(browser): don't resize the webview before the emulator is set up * fix(browser): add reloadAfterLayout helper, only when a page was requested * fix(browser): show "Clear Default" only when a default is saved * fix(browser): refresh default menu items after Set as Default * fix(browser): refresh default menu items after Clear Default

feat: add pinch-to-zoom support to editor

853265a

github-project-automation Bot added this to The Code Board - Acode Oct 9, 2026

github-actions Bot added the enhancement New feature or request label Oct 9, 2026

greptile-apps Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread src/cm/pinchZoom.ts Outdated

Comment thread src/cm/pinchZoom.ts Outdated

Comment thread src/cm/pinchZoom.ts Outdated

Comment thread src/cm/pinchZoom.ts Outdated

fix: address pinch-to-zoom review feedback

1cf50ef

This comment was marked as outdated.

Sign in to view

greptile-apps Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread src/cm/pinchZoom.ts Outdated

fix: address pinch-to-zoom review feedback

2e298a1

This comment was marked as outdated.

Sign in to view

greptile-apps Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread src/cm/pinchZoom.ts Outdated

fix: address pinch-to-zoom review feedback

b771e77

This comment was marked as outdated.

Sign in to view

UnschooledGamer added the community label Oct 9, 2026

UnschooledGamer requested a review from bajrangCoder October 9, 2026 16:49

bajrangCoder reviewed Oct 11, 2026

View reviewed changes

Added pinch to zoom support to editor and made some changes

7dc0871

This comment was marked as outdated.

Sign in to view

greptile-apps Bot reviewed Oct 13, 2026

View reviewed changes

Comment thread src/cm/pinchZoom.ts

Comment thread src/cm/touchSelectionMenu.js Outdated

Fixed corrections for pinchZoom

3be290d

gahanad requested a review from bajrangCoder October 13, 2026 21:32

bajrangCoder reviewed Oct 15, 2026

View reviewed changes

dependabot Bot and others added 4 commits October 16, 2026 02:45

chore: fix Discord badge URL in README (Acode-Foundation#2965)

90f745a

Updated Discord badge URL to a new endpoint.

Added pinchZoom correction

cf1635e

Conversation

gahanad commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Changes Made

Testing

Related Issue

Uh oh!

greptile-apps Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Confidence Score: 5/5

Summary

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

bajrangCoder commented Oct 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

gahanad commented Oct 13, 2026

Uh oh!

bajrangCoder Oct 15, 2026

Choose a reason for hiding this comment

Uh oh!

bajrangCoder commented Oct 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

gahanad commented Oct 9, 2026 •

edited

Loading

greptile-apps Bot commented Oct 9, 2026 •

edited

Loading

bajrangCoder commented Oct 11, 2026 •

edited

Loading