Skip to content

Added IDs to LevelSettingsLayer#171

Open
Flaks45 wants to merge 2 commits into
geode-sdk:mainfrom
Flaks45:LevelSettingsLayer-IDs
Open

Added IDs to LevelSettingsLayer#171
Flaks45 wants to merge 2 commits into
geode-sdk:mainfrom
Flaks45:LevelSettingsLayer-IDs

Conversation

@Flaks45

@Flaks45 Flaks45 commented Jun 15, 2026

Copy link
Copy Markdown

This layer is also used for Start Pos, not too sure why the old commented code was lengthy (since options are in SelectSettingLayer, LevelOptionsLayer, SelectArtLayer, ColorSelectPopup etc etc)

@qimiko

qimiko commented Jun 15, 2026

Copy link
Copy Markdown
Member

the main reason it was so lengthy was bc it was the 2.1 code, where the level settings looked a little bit different. it also separated stuff into separate menus with layouts, whereas this one doesn't. i would say having separate layouts, like the old code, would be nice

@Flaks45

Flaks45 commented Jun 17, 2026

Copy link
Copy Markdown
Author

didn't notice it was 2.1 code, that makes more sense now. About the layouts, 2.1 had more rows and columns that cleanly fit as other menus, and I didn't want to risk creating confusing menus (like the colors being in a layout while most other buttons stayed in another menu). If you have any ideas please lmk, otherwise I will commit again with IDs closer to the 2.1 version and that will be it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants