Skip to content

add geode::Dropdown and replace one-of arrows with dropdown#1931

Open
TheBearodactyl wants to merge 6 commits intogeode-sdk:mainfrom
TheBearodactyl:dropdown
Open

add geode::Dropdown and replace one-of arrows with dropdown#1931
TheBearodactyl wants to merge 6 commits intogeode-sdk:mainfrom
TheBearodactyl:dropdown

Conversation

@TheBearodactyl
Copy link
Copy Markdown
Contributor

No description provided.

@TheBearodactyl TheBearodactyl marked this pull request as ready for review March 16, 2026 03:18
@Cvolton
Copy link
Copy Markdown
Member

Cvolton commented Mar 17, 2026

why does this pr contain so many unrelated formatting changes

@TheBearodactyl
Copy link
Copy Markdown
Contributor Author

because of alk
image

@dankmeme01
Copy link
Copy Markdown
Member

don't think anybody ever runs a formatter in this repo 💀

@TheBearodactyl
Copy link
Copy Markdown
Contributor Author

me, i do
i have format on save enabled
and sadly i can't exactly just unformat it

@HJfod
Copy link
Copy Markdown
Member

HJfod commented Mar 18, 2026

I don't have a strong opinion on including this util in Geode but I strongly disagree with using it anywhere in the loader UI since it is not a UX pattern found elsewhere in vanilla GD

@Alphalaneous
Copy link
Copy Markdown
Contributor

I don't have a strong opinion on including this util in Geode but I strongly disagree with using it anywhere in the loader UI since it is not a UX pattern found elsewhere in vanilla GD

Disagree to an extent. It's easier to know what options are available for one-of. But I also feel it shouldn't be a straight up replacement for the arrows, and instead a control setting

@HJfod
Copy link
Copy Markdown
Member

HJfod commented Mar 18, 2026

I don't have a strong opinion on including this util in Geode but I strongly disagree with using it anywhere in the loader UI since it is not a UX pattern found elsewhere in vanilla GD

Disagree to an extent. It's easier to know what options are available for one-of. But I also feel it shouldn't be a straight up replacement for the arrows, and instead a control setting

I agree from a general UX standpoint this would pretty obviously be better, but I don't want Geode UI to stray away from GD's established UX. Geode already doesn't look like GD, so it should at least feel like it. If we add whatever, then the cognitive load we dump on our potentially non-English-speaking non-computer-savvy 10-year-old children will be too much imo.

Of course this is a very simple component and one they should surely be familiar with from elsewhere, but I want to keep it as a hard rule that Geode may not introduce new UX interactions unless absolutely necessary and if the provided UX adds a considerable benefit.

@Alphalaneous
Copy link
Copy Markdown
Contributor

Personally I feel like that being a hard rule really limits potential. It is a very simple component as you say. There's a certain level of babying the users that just feels unreasonable here.

@Fleeym
Copy link
Copy Markdown
Contributor

Fleeym commented Mar 18, 2026

I personally think one-of settings are extremely annoying to use at the moment, since you have no way of knowing how many values there are until you scroll the entire list through. Having a dropdown would make it so much nicer to use.

Dropdowns are such a widely used pattern on the web that I doubt someone has never heard of one or how it works. In my opinion sticking as close as possible to vanilla should be a balancing act, not a hard rule. If the benefits outweigh the cons by a lot, why not add it?

As long as it's large enough to be easily used on mobile I'd say it's an improvement.

Some screenshots with the dropdown in-game would be useful here.

@HJfod
Copy link
Copy Markdown
Member

HJfod commented Mar 18, 2026

I agree one-of settings are currently pretty ass, but I'd rather add a popup with a list of options and a "Use" button (similar to song selection and other popups in the editor) rather than dropdowns

@Fleeym
Copy link
Copy Markdown
Contributor

Fleeym commented Mar 18, 2026

I agree one-of settings are currently pretty ass, but I'd rather add a popup with a list of options and a "Use" button (similar to song selection and other popups in the editor) rather than dropdowns

That could work, though it's out of the scope of this PR.

@TheBearodactyl
Copy link
Copy Markdown
Contributor Author

I don't have a strong opinion on including this util in Geode but I strongly disagree with using it anywhere in the loader UI since it is not a UX pattern found elsewhere in vanilla GD

Disagree to an extent. It's easier to know what options are available for one-of. But I also feel it shouldn't be a straight up replacement for the arrows, and instead a control setting

will make it into an optional setting, like how you can enable slider control on number settings

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.

6 participants