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

Feat/point and click movement #62

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

Conversation

jga-dev
Copy link
Contributor

@jga-dev jga-dev commented Feb 18, 2023

This PR brings the following changes:

- Fix: Fixes a problem about pointer events blocking in the phaser layer.
- Improve: Allows a better visualization in debug mode.
- Feat: Adds a new way to move by clicking on the place you want to move to. To add this new feature it has been necessary to split the ground and wall layers in 2 different layers. By means of an BFS algorithm it looks for the shortest way taking as premise that it can move through all the ground and that it cannot cross the wall layer.

Hope you find this changes interesting.. I love your project and have many ideas to improve it with new features.

@netlify
Copy link

netlify bot commented Feb 18, 2023

Deploy Preview for skyoffice ready!

Name Link
🔨 Latest commit 03a623e
🔍 Latest deploy log https://app.netlify.com/sites/skyoffice/deploys/63f165b9d683d00008954f06
😎 Deploy Preview https://deploy-preview-62--skyoffice.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jga-dev jga-dev mentioned this pull request Feb 20, 2023
@DaxChen
Copy link
Collaborator

DaxChen commented Feb 26, 2023

Hey @jga-dev, are you still up for resolving the conflicts for this change?

@jga-dev
Copy link
Contributor Author

jga-dev commented Feb 26, 2023

Hi @DaxChen, yes just give me a bit more time. I want to first upload another PR correcting visual problems on mobile and adding the action buttons to events such as sit or view whiteboard. In this PR I want to change the event that starts the movement to double click and improve a few things. I promise it won't be long 👍

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