Skip to content

Update inactive regions to support column granularity - #14667

Merged
Colen Garoutte-Carson (Colengms) merged 2 commits into
mainfrom
dev/coleng/inactive_regions_columns
Aug 13, 2026
Merged

Update inactive regions to support column granularity#14667
Colen Garoutte-Carson (Colengms) merged 2 commits into
mainfrom
dev/coleng/inactive_regions_columns

Conversation

@Colengms

Copy link
Copy Markdown
Contributor

Added column fields.
Broke out a separate line-only object for code folding, which doesn't currently support column offsets.

@8prashant

Copy link
Copy Markdown

Thanks for the update, Colen Garoutte-Carson (@Colengms) .
That makes sense. Since the complete fix needs the native-side changes as well, I'm fine with abandoning this PR #14666 in favor of #14667 .
If any part of this TypeScript-side change or investigation was useful as a reference, would you mind adding a Co-authored-by trailer if appropriate?
Either way, thank you for taking this forward. I'll abandon this PR #14666 .

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates inactive-region decorations to support column-level precision while preserving line-only folding ranges.

Changes:

  • Adds column fields to inactive regions.
  • Introduces a line-only range type for code folding.
  • Applies precise ranges to inactive-region decorations.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Colengms
Colen Garoutte-Carson (Colengms) merged commit 5926619 into main Aug 13, 2026
6 checks passed
@Colengms
Colen Garoutte-Carson (Colengms) deleted the dev/coleng/inactive_regions_columns branch August 13, 2026 20:42
@github-project-automation github-project-automation Bot moved this from Pull Request to Done in cpptools Aug 13, 2026
Sean McManus (sean-mcmanus) added a commit that referenced this pull request Aug 21, 2026
* Validate Yarn lockfile selectors (#14663)

* Fix IntelliSense translation strings. (#14668)

* Fix IntelliSense translation strings.

* Handle localization path casing generically (#14669)

* Update changelog and version for 1.33.8 (#14671)

* Fix configuration path regex escaping (#14674)

* Update inactive regions to support column granularity (#14667)

Co-authored-by: @8prashant (Prashant Kumar Rai)

* Bump the github-actions group with 2 updates (#14678)

Bumps the github-actions group with 2 updates: [github/codeql-action/init](https://github.com/github/codeql-action) and [github/codeql-action/analyze](https://github.com/github/codeql-action).


Updates `github/codeql-action/init` from 4.37.4 to 4.37.6
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@f205ea1...5595cca)

Updates `github/codeql-action/analyze` from 4.37.4 to 4.37.6
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@f205ea1...5595cca)

---
updated-dependencies:
- dependency-name: github/codeql-action/init
  dependency-version: 4.37.6
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
- dependency-name: github/codeql-action/analyze
  dependency-version: 4.37.6
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Upgrade nanoid to 3.3.18 (#14690)

* Fix SimpleCppProject test setup (#14687)

* Update IntelliSense localization strings (#14676)

* Ignore process wait test helper (#14694)

* Retry OneLocBuild dependency installation (#14697)

* Add localized warning for disabled browse database (#14683)

* Add localized browse database warning

* Clarify browse warning translator hint

---------

Co-authored-by: Colen Garoutte-Carson <49173979+Colengms@users.noreply.github.com>

* Harden Linux gdb installation (#14695)

* Fix configuration path diagnostic ranges (#14679)

* Bump the github-actions group with 2 updates (#14698)

Bumps the github-actions group with 2 updates: [github/codeql-action/init](https://github.com/github/codeql-action) and [github/codeql-action/analyze](https://github.com/github/codeql-action).


Updates `github/codeql-action/init` from 4.37.6 to 4.37.7
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@5595cca...ff2f1c6)

Updates `github/codeql-action/analyze` from 4.37.6 to 4.37.7
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@5595cca...ff2f1c6)

---
updated-dependencies:
- dependency-name: github/codeql-action/init
  dependency-version: 4.37.7
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
- dependency-name: github/codeql-action/analyze
  dependency-version: 4.37.7
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update changelog and version for 1.34.0 (#14700)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Colen Garoutte-Carson <49173979+Colengms@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants