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

Adding TapToPlace #31

Merged
merged 4 commits into from
May 12, 2016
Merged

Adding TapToPlace #31

merged 4 commits into from
May 12, 2016

Conversation

jochao
Copy link

@jochao jochao commented May 11, 2016

Adding TapToPlace script which helps place an object on a real surface.

Closes #17

jochao added 2 commits May 11, 2016 15:15
TapToPlace is a quick way to tap on an object, gaze at a different location, and tap again to place object at that location on the surface mesh.
@jochao jochao changed the title Master Adding TapToPlace May 11, 2016

public partial class TapToPlace : MonoBehaviour
{

Copy link
Contributor

@keveleigh keveleigh May 11, 2016

Choose a reason for hiding this comment

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

Please remove this space to match repo style

Mostly formatting changes.  Added info about TapToPlace.unity test scene
in README.
/// and place them on real world surfaces.
/// Put this script on the object you want to be able to move.
/// Users will be able to select objects, gaze elsewhere, and perform the
/// select gesture again to place.
Copy link

Choose a reason for hiding this comment

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

Tap gesture

Changing "select" wording to be "tap" in comments and readme.  Added a
spatialmappingmanager null check.
@NeerajW NeerajW merged commit 17d07d8 into microsoft:master May 12, 2016
david-c-kline pushed a commit that referenced this pull request Oct 19, 2018
…cessRulesFix

input action rules process rules fix
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.

4 participants