ADHOC/replicant-packages_apps_Email
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add account name to temporary sync error UI to help debugging

Browse Source 
* At this point, there are a lot of toasts shown that relate to
  sync failures (e.g. "Unable to connect to server").  To help
  find where they're coming from, add the display name of the
  account
* UX folks: Please consider what this UI should be

Change-Id: Id9b76cd22178727bedaf67ad0b7450edff99a791
This commit is contained in:
Marc Blank 2010-10-19 14:27:21 -07:00
parent 07a91c1655
commit deb51f824e
1 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/com/android/email/activity/MessageListXL.java
View File

@ -18,7 +18,6 @@ package com.android.email.activity;
import com.android.email.Clock;
import com.android.email.Email;
import com.android.email.NotificationController;
import com.android.email.Preferences;
import com.android.email.R;
import com.android.email.RefreshManager;
@ -30,7 +29,6 @@ import com.android.email.provider.EmailContent.Mailbox;
import android.app.ActionBar;
import android.app.Activity;
import android.app.Fragment;
import android.app.LoaderManager.LoaderCallbacks;
import android.content.Context;
import android.content.Intent;
@ -531,8 +529,19 @@ public class MessageListXL extends Activity implements
private class RefreshListener
implements RefreshManager.Listener {
@Override
public void onMessagingError(long accountId, long mailboxId, String message) {
Utility.showToast(MessageListXL.this, message); // STOPSHIP temporary UI
public void onMessagingError(final long accountId, long mailboxId, final String message) {
// STOPSHIP temporary UI
Utility.runAsync(new Runnable() {
@Override
public void run() {
Account account = Account.restoreAccountWithId(mContext, accountId);
String msg = message;
if (account != null) {
msg = account.mDisplayName + ": " + msg;
}
Utility.showToast(MessageListXL.this, msg);
}});
// END STOPSHIP
updateProgressIcon();
}