-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
BBA - Rename Built In BBA to BBA (HLE) #10914
Conversation
I'm not sure this is 100% perfect, but it is a definite improvement. LGTM. |
I find odd that we specify HLE/LLE in parentheses when we don't for DSP. What about just changing Regardless, LGTM. |
_trans("Triforce AM Baseboard"), | ||
_trans("USB Gecko"), | ||
_trans("GCI Folder"), | ||
_trans("Advance Game Port"), | ||
_trans("Broadband Adapter (XLink Kai)"), | ||
_trans("Broadband Adapter (LLE XLink Kai)"), | ||
_trans("Broadband Adapter (tapserver)"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we change this to Broadband Adapter (LLE tapserver)
, so it matches the other ones? LGTM after that.
Or if you go with sepalani's suggestion, skip my suggestion.
All of our BBA options are technically built in, so it made the BBA Built In option kind of confusing as to what it did. So rename it to BBA HLE to make it more clear what it is doing and why it doesn't need a TAP.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
The Broadband Adapter Option names were a bit confusing before, so let's clean them up.