Skip to content

Infinite recursion when showing face chooser. #117

@lpassey

Description

@lpassey

Some of those I play with have complained that they cannot choose a game face. I have encountered this myself on my Windows 11 desktop. Apparently, when the size of the face chooser container (FaceChooserFrame::FaceChooserList) is larger than the available space in the parent FaceChooserFrame it triggers a branch "tooSmall" which is supposed to alter the container size, but which doesn't. The code then triggers doLayout(), which does the size check again, and so on. The solution is to adjust the "wantedSize" Dimension instance to prevent the second and subsequent failures. Increasing the initial size of the parent frame is also desirable.

I have encountered this problem on my Dell desktop (Windows 11), my Lenovo laptop (Windows 11), but not on my HP Pavilion laptop (Ubuntu Linux) or on my wife's Dell laptop (Windows 7). I suspect that on the older systems the size difference test never fails.

Pull request follows.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions