ctm / mb2-doc

Mb2, poker software
https://devctm.com
7 stars 2 forks source link

incorrect semantics for graphical checkboxes #700

Closed ctm closed 2 years ago

ctm commented 2 years ago

Fix it so Call N and Raise N either do what is requested or do nothing.

In the textual interface, Call N means to call any bet of N or less and to fold to any action of N+1 or more. Similarly Raise N means to attempt to either Raise N or to call any action that has the effect of there having been a raise N. Those to behaviors are not what we want in the graphical interface. We want Call N to call any bet of N or less and to do nothing if the action is N+1 or greater. We want Raise to N to mean Raise to N if we're allowed, otherwise do nothing.

I'm labeling this as a bug because the current graphical behavior is not what a PokerStars player would expect. I'm also labeling it as an enhancement because to fix it really will be adding new functionality.

I'm not labeling this easy even though there are some obvious trivial fixes. I am in the middle of working on other bugs though and so I haven't taken the time to convince myself that the trivial fixes are indeed the way to go.

ctm commented 2 years ago

Implemented. Deploying now.