Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): Bump fabric from 4.6.0 to 5.3.0 #6835

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jan 1, 2023

Bumps fabric from 4.6.0 to 5.3.0.

Release notes

Sourced from fabric's releases.

Version 5.2.1

fix(): add eraser to Object state/cache props #7720

Version 5.2.0

Version 5.1.0

Version 5.0.0

Detailed breaking changes available at: here

... (truncated)

Changelog

Sourced from fabric's changelog.

Changelog

[next]

  • chore(TS): migrate text SVG export mixin #8486
  • refactor(TS): animate and AnimationRegistry to classes #8297 BREAKING:
    • return animation instance from animate instead of a cancel function and remove findAnimationByXXX from AnimationRegistry
    • change animateColor signature to match animate, removed colorEasing
  • fix(Object Stacking): 🔙 refactor logic to support Group 🔝
  • chore(TS): migrate Group/ActiveSelection #8455
  • chore(TS): Migrate smaller mixins to classes (dataurl and serialization ) #8542
  • chore(TS): Convert Canvas events mixin and grouping mixin #8519
  • chore(TS): Remove backward compatibility initialize methods #8525
  • chore(TS): replace getKlass utility with a registry that doesn't require full fabricJS to work #8500
  • chore(): use context in static constructors #8522
  • chore(TS): Convert Canvas class #8510
  • chore(TS): Move object classes #8511
  • chore(TS): polish text #8489
  • chore(TS): fix import cycle, extract groupSVGElements #8506
  • chore(TS): permissive Point typings #8434
  • chore(TS): polish files #8488
  • fix(TS): EventSpec recognition #8497
  • chore(): rm dead code #8493
  • fix(scaleObject): handle when scale is 0 to not bug flip #8490
  • chore(TS): migrate StatiCanvas to TS #8485
  • chore(): refactor Object.__uid++ => uid() #8482
  • chore(TS): migrate object mixins to TS #8414
  • chore(TS): migrate filters #8474
  • chore(TS): BaseBrush abstract methods #8428
  • feat(): Add createObjectDefaultControls and createTextboxDefaultControls to create copies of control sets. #8415
  • fix(PatternBrush): getPatternSrc, rm getPatternSrcFunction #8468
  • chore(TS): more FabricObject typing #8405
  • chore(TS): Observable types #8431
  • chore(TS): migrate Group/ActiveSelection #8455
  • fix(TS): migration error of itext key mixin (#8421) #8457
  • chore(TS): migrate text classes/mixins #8421
  • chore(TS): migrate Image #8443
  • chore(TS): migrate Shadow #8462
  • fix(Itext): show incorrect pointer position after scale changed
  • chore(TS): migrate text classes/mixins #8408
  • chore(TS): migrate Collection #8433
  • ci(): Simplify filestats even more #8449
  • chore(TS): migrate filter backends #8403
  • chore(TS): migrate Text classes/mixins #8408
  • chore(TS): migrate Path #8412
  • ci(): remove unwanted build stats (from #8395) #8416
  • chore(TS): migrate Line #8413
  • chore(TS): migrate Polyline/Polygon #8417
  • chore(TS): migrate Rect #8411

... (truncated)

Commits

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
> **Note** > Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Requires JavaScript code changes labels Jan 1, 2023
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/fabric-5.3.0 branch 3 times, most recently from 21a223a to bb0a8d0 Compare January 11, 2023 21:37
@nwalters512
Copy link
Contributor

@nicknytko since you're probably the most familiar with pl-drawing (which AFAIK is the only thing that uses fabric), would you be able to review this PR?

@nwalters512
Copy link
Contributor

@nicknytko friendly bump, if you have the bandwidth!

This was referenced Mar 1, 2023
Copy link
Member

@jonatanschroeder jonatanschroeder left a comment

Choose a reason for hiding this comment

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

A couple more reasons to consider this update.

Comment on lines +9757 to +10850
"jsdom@npm:^19.0.0":
version: 19.0.0
resolution: "jsdom@npm:19.0.0"
Copy link
Member

Choose a reason for hiding this comment

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

yarn.lock Outdated
Comment on lines 14134 to 15748
"tough-cookie@npm:^4.0.0":
version: 4.1.2
resolution: "tough-cookie@npm:4.1.2"
Copy link
Member

Choose a reason for hiding this comment

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

@jonatanschroeder
Copy link
Member

@dependabot rebase

Bumps [fabric](https://github.com/fabricjs/fabric.js) from 4.6.0 to 5.3.0.
- [Release notes](https://github.com/fabricjs/fabric.js/releases)
- [Changelog](https://github.com/fabricjs/fabric.js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/fabricjs/fabric.js/commits/v5.3.0)

---
updated-dependencies:
- dependency-name: fabric
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/fabric-5.3.0 branch from bb0a8d0 to a3eeda3 Compare July 10, 2023 21:27
@nwalters512
Copy link
Contributor

paging @nicknytko again - any chance you can help us determine if this is safe? If not, just let us know and I'll start digging into pl-drawing.

@nicknytko
Copy link
Collaborator

Ah, thanks for the poke! I'll try to give this a look over the weekend.

@jonatanschroeder
Copy link
Member

I checked http://fabricjs.com/v5-breaking-changes, and apparently the only listed breaking change that affects PL code is in Circle.startAngle and Circle.endAngle, which are now in degrees instead of radians (for consistency). All other listed changes affect code that is not used in PL at all.

@nwalters512
Copy link
Contributor

Oof. That's annoying, but it looks like we only have 7 calls to fabric.Circle to audit.

@jonatanschroeder
Copy link
Member

Oof. That's annoying, but it looks like we only have 7 calls to fabric.Circle to audit.

Note, though, that some of those calls also include some code around it that uses the angles to compute things like clickable region using trig functions, so it's not just a matter of changing the values themselves.

@jonatanschroeder
Copy link
Member

Another thing to consider: fabric 6 seems to be in beta, so if we want to avoid duplicating efforts we can hold off on this change until fabric 6 is released in stable and do it all at once.

Copy link
Contributor Author

dependabot bot commented on behalf of github Jul 1, 2024

A newer version of fabric exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Requires JavaScript code changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants