Skip to content

Remove deprecated code - #566

Merged
gfontorbe merged 2 commits into
masterfrom
gfontorbe/remove-deprecated-code
Sep 4, 2026
Merged

gfontorbe merged 2 commits into
masterfrom
gfontorbe/remove-deprecated-code

Conversation

@gfontorbe

Copy link
Copy Markdown
Contributor

Remove deprecated code from the codebase. Cleanup before v2.0

@gfontorbe gfontorbe added this to the v2.0.0 milestone Sep 3, 2026
@gfontorbe
gfontorbe requested a review from spoenemann September 3, 2026 07:22

@spoenemann spoenemann 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.

Thanks!

export class ViewportAnimation extends Animation {

/** @deprecated Use `Math.log(newViewport.zoom / oldViewport.zoom)` instead. */
protected zoomFactor: number;

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.

🤔 should we provide a getter property for this?

@gfontorbe
gfontorbe merged commit 4e79ef1 into master Sep 4, 2026
2 checks passed
spoenemann added a commit that referenced this pull request Sep 9, 2026
The v2.0.0 sections of the four package changelogs now cover everything
merged since v1.4.0: ESM-only packaging (#515), InversifyJS 8 (#561, the
#XXX placeholders resolved), the deprecated-API sweep (#566) with a
per-symbol replacement list, touch and pointer support (#475, #488), the
bug fixes, the strict-mode signature changes, and the npm/OIDC tooling
moves (#534, #549). Headings read "(unreleased)"; the month is set at
release.

sprotty-library gains README.md, CHANGELOG.md, and LICENSE, which the
published package lacked, and is listed in the root changelog index and
README. The AX roadmap records the outcome; the v2 exec plan's
deprecated-API item is closed against #566.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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.

2 participants