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.

Sign up for GitHub

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

Jump to bottom

InputManager support pointer button and wheel #831

Merged
merged 19 commits into from
Jul 5, 2022
Merged

InputManager support pointer button and wheel #831

merged 19 commits into from
Jul 5, 2022
+524 −174

Conversation

cptbtptpbcptdtptp
Copy link
Collaborator

  • Add enabled and disabled interfaces.
  • Support wheel.
  • Supports getting the button type of the pointer.
  • Supports getting pointer moving delta.
  • Optimize code.

@cptbtptpbcptdtptp cptbtptpbcptdtptp added enhancement New feature or request Input Input related functions labels Jun 15, 2022
@cptbtptpbcptdtptp cptbtptpbcptdtptp added this to the 0.8 milestone Jun 15, 2022
@cptbtptpbcptdtptp cptbtptpbcptdtptp self-assigned this Jun 15, 2022
@cptbtptpbcptdtptp cptbtptpbcptdtptp marked this pull request as draft June 16, 2022 06:51
@cptbtptpbcptdtptp
feat: update code

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
GPG key ID: 770F7A9A5AE15600
Verified
Learn about vigilant mode
fdc5792
@GuoLei1990 GuoLei1990 added the high priority High priority issue label Jun 29, 2022
@cptbtptpbcptdtptp cptbtptpbcptdtptp marked this pull request as ready for review June 30, 2022 15:46
@cptbtptpbcptdtptp cptbtptpbcptdtptp changed the title Optimize : InputManager feat : add PointerButtons. Jun 30, 2022
GuoLei1990
GuoLei1990 reviewed Jul 4, 2022
View reviewed changes
packages/core/src/input/InputManager.ts
*/
get pointerMovingDelta(): Readonly<Vector2 | null> {
return this._initialized ? this._pointerManager._movingDelta : null;
}
Copy link
Member

Choose a reason for hiding this comment

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

This is Not a Not a required function, it's very easy for developer to do this themselft. If you want add, I think add a deltaPosition property is better.

packages/core/src/input/InputManager.ts
return this._initialized && this._pointerManager._pointers.length > 0
? this._pointerManager._currentPosition
: null;
}
Copy link
Member

Choose a reason for hiding this comment

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

This property is our raycast point, I need to think a little about how to expose it to developers！

packages/core/src/input/InputManager.ts
} else {
return false;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

If we use mouse and pencil and touch at the same time, pointer button method(isButtonHeldDown、isButtonDown 、isButtonUp ) in InputManager is not enough? Maybe put this method into Pointer.

@GuoLei1990 GuoLei1990 changed the title feat : add PointerButtons. InputManager support point button and wheel Jul 5, 2022
@GuoLei1990 GuoLei1990 changed the title InputManager support point button and wheel InputManager support pointer button and wheel Jul 5, 2022
@GuoLei1990 GuoLei1990 merged commit 6a00825 into galacean:dev/0.8 Jul 5, 2022
@GuoLei1990 GuoLei1990 linked an issue Jul 5, 2022 that may be closed by this pull request
Support get pointer button state for InputManager #782
Closed
GuoLei1990 added a commit to GuoLei1990/galacean-engine that referenced this pull request Jul 11, 2022
@GuoLei1990
Merge branch 'dev/0.8' into dev/asset-file
957bd7f 
* dev/0.8: (28 commits)
  v0.8.0-alpha.3
  v0.8.0-alpha.2
  Use char cache mode for TextRenderer (galacean#837)
  Add basic physics joint component include `FixedJoint`, `SpringJoint`, `HingeJoint`  (galacean#853)
  `InputManager` support pointer button and wheel (galacean#831)
  fix: TextureCube is left-hand,so x need inverse (galacean#855)
  Add SpriteRenderer drawMode property(support simple and sliced)  (galacean#828)
  v0.8.0-alpha.1
  v0.8.0-alpha.0
  Feat(core): add more clear flag item for Camera.clearFlags (galacean#843)
  Fix transform`translate` and `rotate` space bug (galacean#847)
  Refactor `cloneTo` to `copyFrom` for math library and rename `setValue` to `set` (galacean#844)
  optimization camera code (galacean#830)
  feat: physics character controller (galacean#818)
  v0.7.0-beta.7
  refactor: rename (galacean#827)
  Merge latest main (galacean#820)
  fix:   change `region` and `pivot` origin to left-bottom (galacean#809)
  Fix SkyBox render mirror problem (galacean#816)
  feat: add `priority` for renderer to order(galacean#803)
  ...

# Conflicts:
#	lerna.json
#	packages/core/package.json
#	packages/core/src/2d/dynamic-atlas/DynamicTextAtlas.ts
#	packages/core/src/2d/sprite/Sprite.ts
#	packages/core/src/2d/sprite/SpriteMask.ts
#	packages/core/src/2d/sprite/SpriteRenderer.ts
#	packages/core/src/2d/text/TextRenderer.ts
#	packages/core/src/Renderer.ts
#	packages/core/src/Transform.ts
#	packages/core/src/physics/index.ts
#	packages/design/package.json
#	packages/design/src/physics/IPhysics.ts
#	packages/draco/package.json
#	packages/loader/package.json
#	packages/loader/src/SpriteAtlasLoader.ts
#	packages/math/package.json
#	packages/math/src/CollisionUtil.ts
#	packages/math/src/Vector2.ts
#	packages/math/src/Vector3.ts
#	packages/math/src/Vector4.ts
#	packages/oasis-engine/package.json
#	packages/physics-lite/package.json
#	packages/physics-lite/src/LitePhysics.ts
#	packages/physics-physx/package.json
#	packages/physics-physx/src/PhysXPhysics.ts
#	packages/resource-process/package.json
#	packages/resource-process/tsconfig.json
#	packages/rhi-webgl/package.json
#	tests/package.json
gz65555 added a commit that referenced this pull request Jul 11, 2022
@GuoLei1990 @gz65555
@cptbtptpbcptdtptp @singlecoder @luzhuang @zhuxudong @yangfengzzz
Merge latest dev/0.8 to dev/asset-file (#860)
fbbf57a 
* Test: Use Floss and Chai to Replace Jest (#778)

test: Use Floss and Chai to Replace Jest

* refactor: remove toolkit features (#780)

* v0.7.0-beta.4 (#795)

* Fix:local translate (#706)

* fix(transform):local translate

* Add component denpendent decorator (#796)

* feat: add `dependentComponents` decorator

* ci: remove ci on node 12 (#801)

* Update README.md

* Fix Test Version Error (#802)

* ci: remove ci on node 12

* test: fix version

* feat: add `priority` for renderer to order(#803)

* feat(renderer): add `priority` for renderer to order

* Fix SkyBox render mirror problem (#816)

* refactor: fix skybox mirror problem

* fix:   change `region` and `pivot` origin to left-bottom (#809)

* fix:   change `region` and `pivot` origin to left-bottom

* Merge latest main (#820)

* v0.7.0-beta.4

* Update README.md

* Update README.md

* Update README.md

* Add component denpendent decorator (#796) (#807)

* feat: add `dependentComponents` decorator

* fix(Renderer): destroy crash when material is null (#808)

* Improve BlendShape when use attribute mode (#804)

* feat: improve BlendShape when use attribute mode

* v0.7.0-beta.5

* refactor: fix skybox mirror problem

* fix: fixed the bug of animator revert when blendWeight length exceeds 4 (#817)

* fix: fixed the bug of animator revert when blendWeight length exceeds 4

* v0.7.0-beta.6

* refactor: fix shader

* fix(text): fix horizontal and vertical alignment error (#772)

* fix(text): fix horizontal alignment error

Co-authored-by: luzhuang <364439895@qq.com>
Co-authored-by: singlecoder <singlecoder2014@gmail.com>

* refactor: rename (#827)

* v0.7.0-beta.7

* feat: physics character controller (#818)

* feat: add `CharacterController` for physX backend

* optimization camera code (#830)

refactor: optimization camera code

* Refactor `cloneTo` to `copyFrom` for math library and rename `setValue` to `set` (#844)

* feat: refactor `cloneTo` to `copyFrom` for math library
* feat: rename `setValue` to `set` for math library

* Fix transform`translate` and `rotate` space bug (#847)

* fix: transform`translate` and `rotate` space bug
* refactor: opt comments

* Feat(core): add more clear flag item for Camera.clearFlags (#843)

* feat(core): add more clear flag item for Camera.clearFlags

* v0.8.0-alpha.0

* v0.8.0-alpha.1

* Add SpriteRenderer drawMode property(support simple and sliced)  (#828)

* feat: add sprite draw mode(support simple and sliced)

* fix: TextureCube is left-hand,so x need inverse (#855)

* `InputManager` support pointer button and wheel (#831)

* feat: `InputManager` support pointer button and wheel

* Add basic physics joint component include `FixedJoint`, `SpringJoint`, `HingeJoint`  (#853)

* feat: basic physics joint

* Use char cache mode for TextRenderer (#837)

* feat(text): use char cache mode for TextRenderer

* v0.8.0-alpha.2

* v0.8.0-alpha.3

* fix: resolve conflict

* fix: resolve conflict

Co-authored-by: Hu Song <gz65555@gmail.com>
Co-authored-by: AZhan <njktsmshh@163.com>
Co-authored-by: singlecoder <singlecoder2014@gmail.com>
Co-authored-by: luzhuang <364439895@qq.com>
Co-authored-by: zhuxudong <callzhuxudong@163.com>
Co-authored-by: yangfengzzz <yangfengzzz@hotmail.com>
gz65555 added a commit that referenced this pull request Jul 11, 2022
@gz65555 @GuoLei1990
@cptbtptpbcptdtptp @singlecoder @luzhuang @zhuxudong @yangfengzzz
Feature: add script component (#864)
6f1c987 
* fix: use string replace asset type to extend easily

* fix: onEndRender trigger error

* fix: remove ms blob deprecated

* build: update typescript version

* feat: update background mesh

* refactor: add gc ignore

* build: physics use oasis-engine dependency

* bump version

* bump version

* feat: add register and reflection object

* feat: add prefab parser

* fix: resourceManager has't destory all cache

* fix: cache promise is not clear

* bump version

* feat: export prefabloader

* fix: dynamic explansion

* feat: add mipmap generate warn

* feat: add mipmap true

* feat: add pixel buffer

* feat: update engine version

* feat: add type watch

* feat: init all project

* build: remove init oasis

* refactor: use modelmesh encode buffer

* Test: Use Floss and Chai to Replace Jest (#778)

test: Use Floss and Chai to Replace Jest

* refactor: remove console

* refactor: add math serialize

* refactor: add model mesh serialize

* refactor: add scene loader

* refactor: delete schema parser

* refactor: add scene loader

* refactor: add index

* refactor: remove toolkit features (#780)

* v0.7.0-beta.4 (#795)

* refactor: fix types error

* Fix:local translate (#706)

* fix(transform):local translate

* Add component denpendent decorator (#796)

* feat: add `dependentComponents` decorator

* ci: remove ci on node 12 (#801)

* Update README.md

* Fix Test Version Error (#802)

* ci: remove ci on node 12

* test: fix version

* feat: add `priority` for renderer to order(#803)

* feat(renderer): add `priority` for renderer to order

* refactor: remove useless import

* Fix SkyBox render mirror problem (#816)

* refactor: fix skybox mirror problem

* fix:   change `region` and `pivot` origin to left-bottom (#809)

* fix:   change `region` and `pivot` origin to left-bottom

* Merge latest main (#820)

* v0.7.0-beta.4

* Update README.md

* Update README.md

* Update README.md

* Add component denpendent decorator (#796) (#807)

* feat: add `dependentComponents` decorator

* fix(Renderer): destroy crash when material is null (#808)

* Improve BlendShape when use attribute mode (#804)

* feat: improve BlendShape when use attribute mode

* v0.7.0-beta.5

* refactor: fix skybox mirror problem

* fix: fixed the bug of animator revert when blendWeight length exceeds 4 (#817)

* fix: fixed the bug of animator revert when blendWeight length exceeds 4

* v0.7.0-beta.6

* refactor: fix shader

* fix(text): fix horizontal and vertical alignment error (#772)

* fix(text): fix horizontal alignment error

Co-authored-by: luzhuang <364439895@qq.com>
Co-authored-by: singlecoder <singlecoder2014@gmail.com>

* feat: export animation type

* refactor: bump version

* refactor: rename (#827)

* v0.7.0-beta.7

* feat: physics character controller (#818)

* feat: add `CharacterController` for physX backend

* optimization camera code (#830)

refactor: optimization camera code

* Refactor `cloneTo` to `copyFrom` for math library and rename `setValue` to `set` (#844)

* feat: refactor `cloneTo` to `copyFrom` for math library
* feat: rename `setValue` to `set` for math library

* feat: add to Obejct

* feat: remove swc core

* refactor: change toJSON to toObject

* Fix transform`translate` and `rotate` space bug (#847)

* fix: transform`translate` and `rotate` space bug
* refactor: opt comments

* refactor: merge dev/0.8

* refactor: remove console

* Feat(core): add more clear flag item for Camera.clearFlags (#843)

* feat(core): add more clear flag item for Camera.clearFlags

* v0.8.0-alpha.0

* v0.8.0-alpha.1

* feat: update resource process

* Add SpriteRenderer drawMode property(support simple and sliced)  (#828)

* feat: add sprite draw mode(support simple and sliced)

* fix: TextureCube is left-hand,so x need inverse (#855)

* `InputManager` support pointer button and wheel (#831)

* feat: `InputManager` support pointer button and wheel

* Add basic physics joint component include `FixedJoint`, `SpringJoint`, `HingeJoint`  (#853)

* feat: basic physics joint

* Use char cache mode for TextRenderer (#837)

* feat(text): use char cache mode for TextRenderer

* v0.8.0-alpha.2

* v0.8.0-alpha.3

* feat: add script parser

Co-authored-by: gz65555 <kevin.hs@antgroup.com>
Co-authored-by: ChenMo <gl3336563@163.com>
Co-authored-by: AZhan <njktsmshh@163.com>
Co-authored-by: singlecoder <singlecoder2014@gmail.com>
Co-authored-by: luzhuang <364439895@qq.com>
Co-authored-by: zhuxudong <callzhuxudong@163.com>
Co-authored-by: yangfengzzz <yangfengzzz@hotmail.com>
gz65555 added a commit that referenced this pull request Jul 20, 2022
@JujieX @gz65555
@GuoLei1990 @cptbtptpbcptdtptp @singlecoder @luzhuang @zhuxudong @yangfengzzz
Dev/asset file (#876)
c9a5d2c 
* Test: Use Floss and Chai to Replace Jest (#778)

test: Use Floss and Chai to Replace Jest

* refactor: remove toolkit features (#780)

* v0.7.0-beta.4 (#795)

* Fix:local translate (#706)

* fix(transform):local translate

* Add component denpendent decorator (#796)

* feat: add `dependentComponents` decorator

* ci: remove ci on node 12 (#801)

* Update README.md

* Fix Test Version Error (#802)

* ci: remove ci on node 12

* test: fix version

* feat: add `priority` for renderer to order(#803)

* feat(renderer): add `priority` for renderer to order

* Fix SkyBox render mirror problem (#816)

* refactor: fix skybox mirror problem

* fix:   change `region` and `pivot` origin to left-bottom (#809)

* fix:   change `region` and `pivot` origin to left-bottom

* Merge latest main (#820)

* v0.7.0-beta.4

* Update README.md

* Update README.md

* Update README.md

* Add component denpendent decorator (#796) (#807)

* feat: add `dependentComponents` decorator

* fix(Renderer): destroy crash when material is null (#808)

* Improve BlendShape when use attribute mode (#804)

* feat: improve BlendShape when use attribute mode

* v0.7.0-beta.5

* refactor: fix skybox mirror problem

* fix: fixed the bug of animator revert when blendWeight length exceeds 4 (#817)

* fix: fixed the bug of animator revert when blendWeight length exceeds 4

* v0.7.0-beta.6

* refactor: fix shader

* fix(text): fix horizontal and vertical alignment error (#772)

* fix(text): fix horizontal alignment error

Co-authored-by: luzhuang <364439895@qq.com>
Co-authored-by: singlecoder <singlecoder2014@gmail.com>

* refactor: rename (#827)

* v0.7.0-beta.7

* feat: physics character controller (#818)

* feat: add `CharacterController` for physX backend

* optimization camera code (#830)

refactor: optimization camera code

* Refactor `cloneTo` to `copyFrom` for math library and rename `setValue` to `set` (#844)

* feat: refactor `cloneTo` to `copyFrom` for math library
* feat: rename `setValue` to `set` for math library

* Fix transform`translate` and `rotate` space bug (#847)

* fix: transform`translate` and `rotate` space bug
* refactor: opt comments

* Feat(core): add more clear flag item for Camera.clearFlags (#843)

* feat(core): add more clear flag item for Camera.clearFlags

* v0.8.0-alpha.0

* v0.8.0-alpha.1

* Add SpriteRenderer drawMode property(support simple and sliced)  (#828)

* feat: add sprite draw mode(support simple and sliced)

* fix: TextureCube is left-hand,so x need inverse (#855)

* `InputManager` support pointer button and wheel (#831)

* feat: `InputManager` support pointer button and wheel

* Add basic physics joint component include `FixedJoint`, `SpringJoint`, `HingeJoint`  (#853)

* feat: basic physics joint

* Use char cache mode for TextRenderer (#837)

* feat(text): use char cache mode for TextRenderer

* v0.8.0-alpha.2

* v0.8.0-alpha.3

* Optimization performance for `TextRenderer` to update world positions (#861)

* refactor(text): optimization performance for `TextRenderer` to update world positions

* Fix `InputManager` bug (#863)

* fix: `pointer` position bug
* fix: `pointerMovingDelta` value bug
* fix: input `onPointerXX` bug not follow `Camera` cullingMask

* fix(2d): `SpriteRenderer` can not show error (#865)

* v0.8.0-alpha.4

* v0.8.0-alpha.5

Co-authored-by: Hu Song <gz65555@gmail.com>
Co-authored-by: ChenMo <gl3336563@163.com>
Co-authored-by: AZhan <njktsmshh@163.com>
Co-authored-by: singlecoder <singlecoder2014@gmail.com>
Co-authored-by: luzhuang <364439895@qq.com>
Co-authored-by: zhuxudong <callzhuxudong@163.com>
Co-authored-by: yangfengzzz <yangfengzzz@hotmail.com>
luzhuang pushed a commit to luzhuang/engine that referenced this pull request Sep 6, 2022
@cptbtptpbcptdtptp
InputManager support pointer button and wheel (galacean#831)
9e5972a 
* feat: `InputManager` support pointer button and wheel
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GuoLei1990 GuoLei1990

@gz65555 gz65555

@singlecoder singlecoder

@zhuxudong zhuxudong

Assignees

@cptbtptpbcptdtptp cptbtptpbcptdtptp

Labels
enhancement New feature or request high priority High priority issue Input Input related functions
Projects
None yet
Milestone
0.8
Development

Successfully merging this pull request may close these issues.

Support get pointer button state for InputManager
2 participants
@cptbtptpbcptdtptp @GuoLei1990