-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Improve save data dialog #4377
Improve save data dialog #4377
Conversation
Looks good, but I think the wording might need to be changed? New Saved Data > Save Data Personally I think the first option is better as writing "New" twice looks a little bit odd. |
Hmm, indeed. |
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.
The code looks good. I agree with asinine's comment about the grammar though.
Bug fix: don't display new data entry when not asked for Use icon/title provided by the game for the new data entry Display new data entry at the beginning of list when necessary Minor cellSaveData cleanup
Looks good, maybe just change the top line to "New Save Data" instead of "New Saved Data" ? |
It is not a string from code. |
It's not a string from code. |
Don't display new data when unnecessary (bug fix)
Use new data icon/title if provided by the game
Minor cellSaveData cleanup