159356-1702-Extramural / capstone

Capstone Project
Mozilla Public License 2.0
5 stars 5 forks source link

Trade Button behaviour #313

Closed timack closed 6 years ago

timack commented 6 years ago

I found this quite weird how the same button could cause different behaviour, sometimes you'd get three sub buttons (screenshot below), sometimes you'd go direct into interplayer trade, sometime you'd get a help modal.

image

It might make more sense to just disable the unavailable options so the user doesn't get confused - we use the same patter in other parts of the ui - eg. buy dev card / finish turn

image

flukejones commented 6 years ago

Fixed in PR #325