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(GameScreen): add "follow my bot" button #89

Merged

Conversation

aldrin312
Copy link
Contributor

How does this PR impact the user?

Description

  • Refactored zoom functions to be generally used.
  • Combined both zoomIn and zoomOut functions to a single function.
  • Added button to follow player bot and zoom in closely.

Limitations

  • The follow bot function breaks when pressing the follow bot button firstly after loading. This also breaks the other zoom functions. Could not figure out what's causing this. Asking insights for this one.

image

  • I needed to zoom in and zoom out first using mouse wheel for everything to work properly.

Checklist

  • my PR is focused and contains one wholistic change
  • I have added screenshots or screen recordings to show the changes

@yurijmikhalevich yurijmikhalevich changed the title Follow button feat(GameScreen): add "follow my bot" button Dec 2, 2024
@aldrin312
Copy link
Contributor Author

aldrin312 commented Dec 2, 2024

Resolved the conflicts and changed the follow button label.

@aldrin312
Copy link
Contributor Author

Fixed the error that I was getting on first load.

@yurijmikhalevich
Copy link
Member

@aldrin312, thank you for the update 🙌 I'll take another look later this week.

@aldrin312
Copy link
Contributor Author

aldrin312 commented Dec 8, 2024

Fixed the issue.
It should follow the appropriate bot now.
Follow button should say "follow my bot" if not following player bot and "stop following my bot" if it's following.

Demo:

aldrin312 and others added 4 commits December 17, 2024 17:20
…ngs#92)

## How does this PR impact the user?



https://github.com/user-attachments/assets/89f97e3f-75d4-414b-b102-0c86fd24eda8


## Description

- Added `/api/state` to public paths to make current game publicly
accessible.
- Updated the login screen to display the login form on the left and the
current game on the right (reused the code from `index.vue`).
- Moved the login form to a separate component
(`/components/LoginForm.vue`).
- Increased the timeout for the login e2e tests to 60 seconds, as 30
seconds timeout was insufficient for the tests to complete.

## Limitations

The game screen takes a couple of seconds to load.

## Checklist

- [x] my PR is focused and contains one wholistic change
- [x] I have added screenshots or screen recordings to show the changes
Copy link
Member

@yurijmikhalevich yurijmikhalevich left a comment

Choose a reason for hiding this comment

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

Thank you! It works great! Let's clean it up a bit more and merge.

You or I can take a look at what's up with appRef when you try to move the zoom-in logic into a watch callback in a follow-up.

@aldrin312
Copy link
Contributor Author

aldrin312 commented Dec 19, 2024

I'll try looking into the appRef issue more. Cause it might become in issue again later.

Copy link
Member

@yurijmikhalevich yurijmikhalevich left a comment

Choose a reason for hiding this comment

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

Thank you!

@yurijmikhalevich yurijmikhalevich merged commit 68ee712 into move-fast-and-break-things:main Dec 21, 2024
2 checks passed
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