-
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
UI improvements to Add AR Code and Add Patch windows. #2456
Conversation
@dolphin-emu-bot rebuild |
Please use a more descriptive commit message/PR title. |
@@ -178,6 +179,7 @@ void CARCodeAddEdit::UpdateTextCtrl(ActionReplay::ARCode arCode) | |||
} | |||
else | |||
{ | |||
EditCheatCode->SetValue(_("Insert Encrypted or Decrypted code here...")); | |||
//EditCheatCode->SetValue(_("Insert Encrypted or Decrypted code here...")); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
f66aa16
to
0a32c8d
Compare
@dolphin-emu-bot rebuild |
@@ -30,7 +30,7 @@ CARCodeAddEdit::CARCodeAddEdit(int _selection, std::vector<ActionReplay::ARCode> | |||
Bind(wxEVT_BUTTON, &CARCodeAddEdit::SaveCheatData, this, wxID_OK); | |||
|
|||
ActionReplay::ARCode tempEntries; | |||
wxString currentName = _("Insert name here..."); | |||
wxString currentName = _(""); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
aea561e
to
321b55c
Compare
I think renaming the pr title to what you're actually changing would be nice. |
Text fields no longer have redundant text. Blank text variables no longer have redundant translation syntax. Removed redundant else condition. Fixed bug with cheat name deleting itself on edit.
@dolphin-emu-bot rebuild |
LGTM |
UI improvements to Add AR Code and Add Patch windows.
I removed the text from both textfields and added a "Code" label to describe the cheatcode field. Much cleaner.