voxeltycoon / issues

18 stars 4 forks source link

Copying vehicles function in depot window overrides behavior of vehicle sell button #727

Closed hairintd closed 3 years ago

hairintd commented 3 years ago

Describe the bug Car in depot shows as a tile with it's icon and buttons for sell it or change settings. Copy button of the depot allow you to hover this tile and make a copy of this car in one click, which from one side is a great feature making life easier, and from the second - you get two cars, when you actually try to sell a car.

Steps to reproduce

  1. Have a depot window opened, have a one car inside this depo. You want to copy a car so you have to press copy button. 20210509011324_1

  2. At this point you find out, that you have to sell existing car first (i.e. not enough money to buy a copy of new one). So you point to a sell button on the car tile in the depo and press it. 20210509011328_1

  3. You get 2 old cars instead of zero (if you have enough money for old one). 20210509011332_1

If you exclude actual buttons (settings, sell on this tile in depot window) from being a target for copy button - it would be great i think.

Build version 0.85.2.1 steam

Platform Windows 10

Reproducibility every time

Logs/saves