Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Merge pull request #11181 from crdlc/bug-hide-status-from-import-cenc…
Browse files Browse the repository at this point in the history
…eled

Bug 898287 - [Cancel Import] Don't show the status UI component after canceling
  • Loading branch information
Cristian Rodriguez committed Jul 26, 2013
2 parents a3d7268 + 4e3a598 commit b67a4e7
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 10 deletions.
16 changes: 10 additions & 6 deletions apps/communications/contacts/js/contacts_settings.js
Expand Up @@ -412,9 +412,11 @@ contacts.Settings = (function() {
Contacts.navigation.home();
if (importedContacts !== 0) {
window.importUtils.setTimestamp('sim');
Contacts.showStatus(_('simContacts-imported3', {
n: importedContacts
}));
if (!cancelled) {
Contacts.showStatus(_('simContacts-imported3', {
n: importedContacts
}));
}
}
}, DELAY_FEEDBACK);

Expand Down Expand Up @@ -505,9 +507,11 @@ contacts.Settings = (function() {
window.importUtils.setTimestamp('sd');
resetWait(wakeLock);
Contacts.navigation.home();
Contacts.showStatus(_('memoryCardContacts-imported3', {
n: importedContacts
}));
if (!cancelled) {
Contacts.showStatus(_('memoryCardContacts-imported3', {
n: importedContacts
}));
}
}, DELAY_FEEDBACK);
});
}
Expand Down
2 changes: 1 addition & 1 deletion apps/communications/contacts/js/importer_ui.js
Expand Up @@ -695,7 +695,7 @@ if (typeof window.importer === 'undefined') {
parent.postMessage({
type: 'window_close',
data: '',
message: _('friendsUpdated', {
message: cancelled ? '' : _('friendsUpdated', {
numFriends: numFriends
})
}, targetApp);
Expand Down
6 changes: 4 additions & 2 deletions apps/communications/ftu/js/memcard_manager.js
Expand Up @@ -83,8 +83,10 @@ var SdManager = {

UIManager.navBar.removeAttribute('aria-disabled');
utils.overlay.hide();
utils.status.show(
_('memoryCardContacts-imported3', {n: importedContacts}));
if (!cancelled) {
utils.status.show(
_('memoryCardContacts-imported3', {n: importedContacts}));
}

}, DELAY_FEEDBACK);
});
Expand Down
5 changes: 4 additions & 1 deletion apps/communications/ftu/js/sim_manager.js
Expand Up @@ -412,7 +412,10 @@ var SimManager = {
if (importedContacts !== 0) {
window.importUtils.setTimestamp('sim');
SimManager.alreadyImported = true;
utils.status.show(_('simContacts-imported3', {n: importedContacts}));
if (!cancelled) {
utils.status.show(_('simContacts-imported3',
{n: importedContacts}));
}
}
}, DELAY_FEEDBACK);

Expand Down

0 comments on commit b67a4e7

Please sign in to comment.