Skip to content

Conversation

@puxiao
Copy link
Contributor

@puxiao puxiao commented Jul 3, 2021

Related issue: null

Description

emm....
Even if you don’t add this parameter, it can still be executed normally.

@Mugen87
Copy link
Collaborator

Mugen87 commented Jul 3, 2021

Yes, it is implicitly defined. But it is of course more readable when event is defined in the signature.

@Mugen87 Mugen87 added this to the r131 milestone Jul 3, 2021
@mrdoob mrdoob modified the milestones: r131, r130 Jul 4, 2021
@mrdoob mrdoob merged commit 21e7244 into mrdoob:dev Jul 4, 2021
@mrdoob
Copy link
Owner

mrdoob commented Jul 4, 2021

Thanks!

@puxiao puxiao deleted the patch-3 branch July 6, 2021 00:53
ThorstenBux added a commit to ThorstenBux/three.js that referenced this pull request Jul 6, 2021
…l-placement-usdz

* commit 'd0340e3a147e290fa86d14bc3ed97d8e1c20602e': (79 commits)
  r130 (bis)
  Revert "Update revision (mrdoob#22087)" (mrdoob#22090)
  Update WebXR Layers example to compare quality. (mrdoob#22083)
  Update revision (mrdoob#22087)
  Updated examples builds
  Updated builds
  USDZExporter: Added map transparency support (mrdoob#22086)
  DragControls: added onPointerDown parameters (mrdoob#22084)
  Examples: Clean up.
  FBXLoader: Matrix4 code clean up.
  USDZExporter: Log a warning when matrixWorld has a negative scale.
  Docs: Improved AnimationMixer page. (mrdoob#22076)
  WebGLTextures: Fix mips gen for 3D render targets. (mrdoob#22072)
  Fix regression in WebXR (mrdoob#22074)
  Revert "TransformControls: Clean up."
  r130
  Examples: Clean up
  Editor: Cache network responses in sw.js.
  Updated builds
  Updated examples builds
  ...

# Conflicts:
#	examples/jsm/exporters/USDZExporter.js
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.

3 participants