Skip to content

feat(editor): add configurable word wrap indentation - #2886

Merged
bajrangCoder merged 3 commits into
mainfrom
wrap-indent-setting
Sep 12, 2026
Merged

bajrangCoder merged 3 commits into
mainfrom
wrap-indent-setting

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added the translations Anything related to Translations Whether a Issue or PR label Sep 12, 2026
@greptile-apps

greptile-apps Bot commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the latest changes preserve the wrapping behavior while eliminating the previously noted full-line scan.

Findings

  1. P2 Locale Strings Remain English ▶

Summary

  • Supports no indentation, matching indentation, and one- or two-level additional indentation.
  • Reconfigures active editors when the wrapping-indentation setting changes.
  • Preserves a bounded fast path for large unindented lines.
  • Adds setting labels, locale keys, typings, and unit coverage.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
    A[Editor setting: wrappingIndent] --> B[EditorManager wrap compartment]
    B --> C[indentedLineWrapping mode]
    C --> D[Wrapping-indent facet]
    D --> E[Visible-line decoration]
    E --> F[Continuation padding and text indent]
Loading

Reviews (2) · Last reviewed commit: "fix"

Comment thread src/cm/indentedLineWrapping.ts Outdated
Comment thread src/lang/ar-ye.json
@bajrangCoder

This comment was marked as outdated.

@bajrangCoder
bajrangCoder added this pull request to the merge queue Sep 12, 2026
Merged via the queue into main with commit c479364 Sep 12, 2026
11 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Sep 12, 2026
@bajrangCoder
bajrangCoder deleted the wrap-indent-setting branch September 12, 2026 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translations Anything related to Translations Whether a Issue or PR

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant