GuzhavinAleksey/Vmeda.Online
1
0
Fork 0
forked from CIT/Vmeda.Online
Code Pull Requests Activity

MOBILE-2567 notifications: Add spiner on notifications mark all as read

Browse Source
This commit is contained in:
Pau Ferrer Ocaña 2018-09-05 12:04:23 +02:00
parent 0fbb1e2fd2
commit 307aab820a
2 changed files with 27 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/addon/notifications/pages/list/list.html
View File

@ -9,10 +9,13 @@
</ion-refresher>
<core-loading [hideUntil]="notificationsLoaded">
<div padding *ngIf="canMarkAllNotificationsAsRead">
<button ion-button block (click)="markAllNotificationsAsRead()" color="light" icon-start>
<button ion-button block (click)="markAllNotificationsAsRead()" color="light" icon-start *ngIf="!loadingMarkAllNotificationsAsRead">
<core-icon name="fa-check"></core-icon>
{{ 'addon.notifications.markallread' | translate }}
</button>
<button ion-button block color="light" icon-start *ngIf="loadingMarkAllNotificationsAsRead">
<ion-spinner></ion-spinner>
</button>
</div>
<ion-card *ngFor="let notification of notifications">
<ion-item>

36
src/addon/notifications/pages/list/list.ts
View File

@ -37,6 +37,7 @@ export class AddonNotificationsListPage {
notificationsLoaded = false;
canLoadMore = false;
canMarkAllNotificationsAsRead = false;
loadingMarkAllNotificationsAsRead = false;
protected readCount = 0;
protected unreadCount = 0;
@ -118,15 +119,6 @@ export class AddonNotificationsListPage {
this.canLoadMore = true;
}
// Check if mark all notifications as read is enabled and there are some to read.
if (this.notificationsProvider.isMarkAllNotificationsAsReadEnabled()) {
promises.push(this.notificationsProvider.getUnreadNotificationsCount().then((unread) => {
this.canMarkAllNotificationsAsRead = unread > 0;
}));
} else {
this.canMarkAllNotificationsAsRead = false;
}
return Promise.all(promises).then(() => {
// Mark retrieved notifications as read if they are not.
this.markNotificationsAsRead(unread);
@ -141,15 +133,17 @@ export class AddonNotificationsListPage {
* Mark all notifications as read.
*/
markAllNotificationsAsRead(): void {
this.notificationsProvider.markAllNotificationsAsRead().then(() => {
this.loadingMarkAllNotificationsAsRead = true;
this.notificationsProvider.markAllNotificationsAsRead().catch(() => {
// Omit failure.
}).finally(() => {
const siteId = this.sitesProvider.getCurrentSiteId();
this.eventsProvider.trigger(AddonNotificationsProvider.READ_CHANGED_EVENT, null, siteId);
this.notificationsProvider.getUnreadNotificationsCount().then((unread) => {
this.canMarkAllNotificationsAsRead = unread > 0;
this.loadingMarkAllNotificationsAsRead = false;
});
}).catch(() => {
// Omit failure.
});
}
@ -159,12 +153,14 @@ export class AddonNotificationsListPage {
* @param {any[]} notifications Array of notification objects.
*/
protected markNotificationsAsRead(notifications: any[]): void {
let promise;
if (notifications.length > 0) {
const promises = notifications.map((notification) => {
return this.notificationsProvider.markNotificationRead(notification.id);
});
Promise.all(promises).catch(() => {
promise = Promise.all(promises).catch(() => {
// Ignore errors.
}).finally(() => {
this.notificationsProvider.invalidateNotificationsList().finally(() => {
@ -172,7 +168,21 @@ export class AddonNotificationsListPage {
this.eventsProvider.trigger(AddonNotificationsProvider.READ_CHANGED_EVENT, null, siteId);
});
});
} else {
promise = Promise.resolve();
}
promise.finally(() => {
// Check if mark all notifications as read is enabled and there are some to read.
if (this.notificationsProvider.isMarkAllNotificationsAsReadEnabled()) {
this.loadingMarkAllNotificationsAsRead = true;
return this.notificationsProvider.getUnreadNotificationsCount().then((unread) => {
this.canMarkAllNotificationsAsRead = unread > 0;
this.loadingMarkAllNotificationsAsRead = false;
});
}
this.canMarkAllNotificationsAsRead = false;
});
}
/**