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

Rename OptionButton to DropDownList #1998

Open
madmiraal opened this issue Dec 17, 2020 · 7 comments
Open

Rename OptionButton to DropDownList #1998

madmiraal opened this issue Dec 17, 2020 · 7 comments
Labels
breaks compat Proposal will inevitably break compatibility topic:gui
Milestone

Comments

@madmiraal
Copy link

Describe the project you are working on

Godot engine

Describe the problem or limitation you are having in your project

As originally mentioned in godotengine/godot#6558, the OptionButton is actually a Drop-down List.

Describe the feature / enhancement and how it helps to overcome the problem or limitation

Rename OptionButton to DropDownList.

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

As above.

If this enhancement will not be used often, can it be worked around with a few lines of script?

N/A

Is there a reason why this should be core and not an add-on in the asset library?

N/A

@Calinou Calinou added this to the 4.0 milestone Dec 17, 2020
@Calinou
Copy link
Member

Calinou commented Dec 17, 2020

It's still a button technically, so I'd rename it to DropdownButton instead. (I think DropDownButton looks kind of ugly.)

@madmiraal
Copy link
Author

It's still a button technically, so I'd rename it to DropdownButton instead. (I think DropDownButton looks kind of ugly.)

It's only a Button, because, within the engine, it inherits Button, but it's a Drop-down list. Note: Checkbox also inherits Button, but it's not called CheckboxButton. I think it should be called what it is.

@groud
Copy link
Member

groud commented Dec 17, 2020

Hmm, this is not correct to me. For me, the dropdown list is the list that pops up, not the button itself.
I believe the current naming is better.

@Calinou Calinou added the breaks compat Proposal will inevitably break compatibility label May 11, 2022
@Mickeon
Copy link

Mickeon commented Aug 21, 2022

It's been 2 years since this proposal, but my vote still goes to DropdownButton mostly because it extends Button. There's no need to be accurate to HTML standards in this case, because it's clear on all users coming to Godot what DropdownButton does simply because of namesake. It's a Button that, when pressed, brings up a Dropdown list.

@YuriSizov
Copy link
Contributor

There doesn't seem to be a lot of support for the OP, though there are many likes to Calinou's counter-suggestion. Not sure if this should be considered as "We vote for rename" or as "If we were to rename, this is a better idea".

Be that as it may, I'm not convinced this is actually helpful anymore. I think the Option qualifier explains better that this button provides a way to select from one of the options. Dropdown is vague and non-specific, it describes how it looks rather than what it does. Even if other UI libraries use that quite often, I don't think it's a major win to rename here.

Of course, this can be bikeshed to death too. We now have a proposal to add a carousel button which is essentially the same but with a different interaction mechanic. So maybe naming this one after its interaction mechanic is not that bad, etc, etc. Personally, I'd go the "no consensus" route here.

@akien-mga
Copy link
Member

We discussed this proposal in a PR review meeting and agreed on the DropdownButton proposal, to keep the inheritance of Button clear (and as @groud pointed it out the "list" part can be understood as the PopupMenu that shows up when you click the button, not the button itself).

Won't die on that hill but the main consensus seems to be "Dropdown" without capitalizing "down" (it's not clearly established in the codebase whether compound names with an hyphen should have both terms capitalized or not).

@Calinou
Copy link
Member

Calinou commented Jun 12, 2023

We discussed this proposal in a PR review meeting and agreed on the DropdownButton proposal

Note that if we decide to add carousel-style functionality to OptionButton, then the name "DropdownButton" will no longer be applicable for that particular style of UI control.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaks compat Proposal will inevitably break compatibility topic:gui
Projects
None yet
7 participants