LEGUP icon indicating copy to clipboard operation
LEGUP copied to clipboard

[BUG] Skycrapers Cell For Number Case Rule

Open willyoung366 opened this issue 1 year ago • 1 comments

Description

In Skyscrapers, if you select Cell for Number case rule, a popup tells you to select the number to be added. This popup has a cancel button, but it does not work. When you press cancel, the popup does not go away.

Steps to Reproduce

  1. Open 4x4 skyscrapers easy1 (I think this bug exists on all boards though)
  2. Select case rules
  3. Select cell for number
  4. Select cancel (the popup won't go away)

Expected Behavior

I expected the popup to go away when you press cancel

Screenshots

{7F68209F-4BBF-46B9-8F48-EFD1B958C2C7}

Puzzle Files

No response

Additional Context

No response

willyoung366 avatar Oct 29 '24 20:10 willyoung366

I looked into it a little bit; The case rule requires a number to fill in the row or column with, so there is a popup for the number to add. The popup window is in a do-while loop and waits until the number is set to something before moving on, which explains why the window doesn't close. It's in the getCaseBoard function within /skyscrapers/rules/CellForNumberCaseRule. However, if you cancel without selecting a number, the engine throws an error because there is no way to cancel a case rule after selecting it. Theoretically, if there is a way to tell the program to exit case rule mode, it would resolve the issue.

jadeandtea avatar Oct 29 '24 21:10 jadeandtea