Skip to content
This repository has been archived by the owner on May 9, 2022. It is now read-only.

Issue #6: String resource conversion #7

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Issue #6: String resource conversion #7

wants to merge 1 commit into from

Conversation

jouva
Copy link

@jouva jouva commented Mar 21, 2015

Converted UI displayed strings to string resources. Resolves #6

Do not merge until this is loaded with an appropriate IDE to generate related R files as well

Notes

  • This only handles parts that were obviously UI to me. There are some thrown Exceptions that have human readable strings in updater classes and elsewhere as well as calls named noalert and error. I am unsure that they are displayed to the user.
  • This does not cover any Logging calls
  • I'm on a Mac and I seem to be inserting or removing ^M and cannot tell which line endings you prefer.

Please let me know if any of these are wrong and I will correct this if necessary

@@ -105,7 +105,7 @@ public void populateViewHolder(Holder holder) {
// Redo name
String name = getName();
if (getOpenStatus() == RestaurantOpenStatus.Closed) {
name += " [Closed]";
name += getString(R.string.brackets_closed);
}

holder.name.setText(name);
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Below here is dollars += "D" single character for hasDelivery(). Did you want this as a string resource as well?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably should be, yes.

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

Successfully merging this pull request may close these issues.

2 participants