Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable resizing of lobby character editor #32313

Conversation

eoineoineoin
Copy link
Contributor

@eoineoineoin eoineoineoin commented Sep 19, 2024

About the PR

Real trivial bugfix. When attempting to close the character editor, I misclicked and accidentally resized the character editor.

Once the editor has been resized, the container remembers the sizes until application restart, and when the user attempts to manually restore the size, it makes a mess of the main lobby screen.

There's no need for these controls to be user-resizable anyway, so just disable that.

Why / Balance

Technical details

Media

Old behaviour; character editor remembers a bad size and is broken whenever you open it:
2024-09-19_21-45

Lobby after you attempt to resize the character editor back to what it was previously:
2024-09-19_21-45_1

Bonus, stops this "resizable" mouse cursor from appearing, even though the panel couldn't be resized:

2024-09-19_21-51

Requirements

Breaking changes

Changelog

@github-actions github-actions bot added Changes: UI Can be reviewed or fixed by people who are knowledgeable with UI design No C# For things that don't need code. labels Sep 19, 2024
@lzk228
Copy link
Contributor

lzk228 commented Sep 19, 2024

does it have something with #27650 ?

@metalgearsloth metalgearsloth merged commit a9b5e39 into space-wizards:master Sep 20, 2024
11 checks passed
Ilya246 pushed a commit to Ilya246/space-station-14 that referenced this pull request Oct 7, 2024
No resizing character editor

Co-authored-by: Eoin Mcloughlin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: UI Can be reviewed or fixed by people who are knowledgeable with UI design No C# For things that don't need code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants