serge-web / serge

Serious Gaming, Evolved - web interface
https://sites.google.com/deepbluec.com/serge/
Apache License 2.0
14 stars 4 forks source link

Two icon drag/drop issues #784

Closed IanMayo closed 10 months ago

IanMayo commented 3 years ago

We have two issues related to moving platforms into / onto others.

There are two actions:

Both are only allowed when the platforms are in the same cell. This bit is working well.

I've modified your implementation - so there is a more green shade when the operation is allowable.

A new requirement means we should not let a Task Group host another platform. The task group is just the name of an organisation - so you can't put a helicopter onto it, for example.

In this example, SSN2 cannot be placed onto SSN3. The drop-target is shown in Red. But, if I still drop it, it gets added to CTF-300. If something is dropped on a disabled drop-zone, nothing should happen.

screenshot

Second issue.

I drag SSN1 over CTF 300 . I should be allowed to drop it onto CTF-300 (comprising), but not to the Move onto drop-zone. The move onto drop-zone is shown as red. But, I can still drop onto it. When I do, it gets added as comprising. I think that nothing should happen.

screenshot

The above interactions can be produced in this deployment: https://serge-demo-task-group-d-ekn98z.herokuapp.com/?wargame=wargame-koiquada&access=pkoiqzx9e

The deployment is from this branch: demo_task_group_drag

IanMayo commented 3 years ago

Closed in #813

IanMayo commented 3 years ago

Issue #813 not closed.