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

Selecting trains requires you to click on top of the track it is on, not the train itself #101

Open
mpomery opened this issue Jul 22, 2020 · 1 comment

Comments

@mpomery
Copy link
Contributor

mpomery commented Jul 22, 2020

In this image clicking on the purple train where the pointer currently is does not select the train.
Same with clicking on the front of the red and rightmost blue trains.

image

Clicking where the pointer is now selects the rightmost blue train even though the pointer is not above the train
image
Hitbox shown:
image

@mpomery
Copy link
Contributor Author

mpomery commented Jul 23, 2020

From Discord:

MitchPommers: Thinking about improving train clickability. Options:

  • Update GameBoard.GetMovableAt to find the train in it's current+it's next cell
  • Send PointerTool the x+y coords of the pointer then find trains where the pointer falls within the radius
  • Make PointerTool use draggable instead (feels hacky)
  • Stop trains being directly clickable and instead create a list of trains you can select from

davidwengier: I think this (Stop trains being directly clickable and instead create a list of trains you can select from), or possibly just removing clicking altogether
davidwengier: but as a "game", clicking on a specific train seems unnecessary

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

No branches or pull requests

1 participant