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 "Menu" to "MenuItem" #67

Open
maxmamis opened this issue Jun 27, 2017 · 2 comments
Open

Rename "Menu" to "MenuItem" #67

maxmamis opened this issue Jun 27, 2017 · 2 comments

Comments

@maxmamis
Copy link
Contributor

The name Menu is confusing — it sounds like a screenful of options. Most classes/protocols/etc in Yoshi ending with the word "Menu" actually represent individual menu items, I.E. cells. YoshiGenericMenu should be YoshiGenericMenuItem, etc.

@maxmamis
Copy link
Contributor Author

maxmamis commented Jun 27, 2017

The existence of YoshiTableViewMenu complicates things, because it already has an associated protocol called YoshiTableViewMenuItem. On the other hand, that name itself is confusing, because it's a very specific kind of table view (allowing you to select one of several options) and that's not reflected by the name — in fact, it being a table view is really just an implementation detail, not the primary characteristic. Here's what I propose:

YoshiTableViewMenu becomes YoshiPickerMenuItem
YoshiTableViewMenuItem becomes YoshiPickerOption

@ghvg1313
Copy link
Contributor

These names are somewhat inherited from 1.0 and gets expanded, for backward-compatibility concern the name will stay the same in the foreseeable future, we may change the name in the next major breaking update.

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

No branches or pull requests

2 participants