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

Two Traders ended up in the same location in trading zone #153

Open
StStep opened this issue Oct 12, 2022 · 1 comment
Open

Two Traders ended up in the same location in trading zone #153

StStep opened this issue Oct 12, 2022 · 1 comment
Assignees
Labels
🐛 bug 🐛 Something isn't working 🔧 tweak 🔧 Not a bug, not really a request, but a tweak is necessary

Comments

@StStep
Copy link

StStep commented Oct 12, 2022

I was only able to select one of them because the other was being obscured.

@SerialKicked SerialKicked self-assigned this Oct 12, 2022
@SerialKicked SerialKicked added 🐛 bug 🐛 Something isn't working 🔧 tweak 🔧 Not a bug, not really a request, but a tweak is necessary labels Oct 12, 2022
@SerialKicked
Copy link
Member

SerialKicked commented Oct 12, 2022

Acknowledged.

Behavior Fix:

  • On trader update. If not moving, check if other trader on same tile. If there's one, check if it's running a move order. If not, move to another tile in the zone (so 2 traders don't move at random only to possibly end up on the same tile again). Issue: none.

UI Fix:

  • On select click. If 2 target are within X pixels of the click, and that one of them was already selected, switch to the other. Might also fix select issues on world map and shit. Issue: probably doesn't cover all cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 🐛 Something isn't working 🔧 tweak 🔧 Not a bug, not really a request, but a tweak is necessary
Projects
None yet
Development

No branches or pull requests

2 participants