Merge pull request #2805 from dpalou/MOBILE-3320

Mobile 3320

src/addons/calendar/calendar-lazy.module.ts

@@ -18,7 +18,7 @@ import { Route, RouterModule, ROUTES, Routes } from '@angular/router';
 import { buildTabMainRoutes } from '@features/mainmenu/mainmenu-tab-routing.module';
 
 export const AddonCalendarEditRoute: Route = {
-    path: 'edit',
+    path: 'edit/:eventId',
     loadChildren: () =>
         import('@/addons/calendar/pages/edit-event/edit-event.module').then(m => m.AddonCalendarEditEventPageModule),
 };

src/addons/calendar/pages/day/day.page.ts

@@ -553,10 +553,9 @@ export class AddonCalendarDayPage implements OnInit, OnDestroy {
     openEdit(eventId?: number): void {
         const params: Params = {};
 
-        if (eventId) {
+        if (!eventId) {
-            params.eventId = eventId;
-        } else {
             // It's a new event, set the time.
+            eventId = 0;
             params.timestamp = moment().year(this.year).month(this.month - 1).date(this.day).unix() * 1000;
         }
 
@@ -564,7 +563,7 @@ export class AddonCalendarDayPage implements OnInit, OnDestroy {
             params.courseId = this.filter.courseId;
         }
 
-        CoreNavigator.navigateToSitePath('/calendar/edit', { params });
+        CoreNavigator.navigateToSitePath(`/calendar/edit/${eventId}`, { params });
     }
 
     /**

src/addons/calendar/pages/edit-event/edit-event.page.ts

@@ -132,7 +132,7 @@ export class AddonCalendarEditEventPage implements OnInit, OnDestroy, CanLeave {
      * Component being initialized.
      */
     ngOnInit(): void {
-        this.eventId = CoreNavigator.getRouteNumberParam('eventId');
+        this.eventId = CoreNavigator.getRouteNumberParam('eventId') || undefined;
         this.courseId = CoreNavigator.getRouteNumberParam('courseId') || 0;
         this.title = this.eventId ? 'addon.calendar.editevent' : 'addon.calendar.newevent';
 

src/addons/calendar/pages/event/event.page.ts

@@ -424,7 +424,7 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
      * Open the page to edit the event.
      */
     openEdit(): void {
-        CoreNavigator.navigateToSitePath('/calendar/edit', { params: { eventId: this.eventId } });
+        CoreNavigator.navigateToSitePath(`/calendar/edit/${this.eventId}`);
     }
 
     /**

src/addons/calendar/pages/index/index.page.ts

@@ -351,15 +351,13 @@ export class AddonCalendarIndexPage implements OnInit, OnDestroy {
      */
     openEdit(eventId?: number): void {
         const params: Params = {};
+        eventId = eventId || 0;
 
-        if (eventId) {
-            params.eventId = eventId;
-        }
         if (this.filter.courseId) {
             params.courseId = this.filter.courseId;
         }
 
-        CoreNavigator.navigateToSitePath('/calendar/edit', { params });
+        CoreNavigator.navigateToSitePath(`/calendar/edit/${eventId}`, { params });
     }
 
     /**

src/addons/calendar/pages/list/list.html

@@ -20,7 +20,6 @@
     </ion-toolbar>
 </ion-header>
 <ion-content>
-    <core-split-view>
     <ion-refresher slot="fixed" [disabled]="!eventsLoaded" (ionRefresh)="doRefresh($event.target)">
         <ion-refresher-content pullingText="{{ 'core.pulltorefresh' | translate }}"></ion-refresher-content>
     </ion-refresher>
@@ -93,5 +92,4 @@
             <ion-icon name="fas-plus" aria-hidden="true"></ion-icon>
         </ion-fab-button>
     </ion-fab>
-    </core-split-view>
 </ion-content>

src/addons/calendar/pages/list/list.page.ts

@@ -29,7 +29,6 @@ import { CoreSites } from '@services/sites';
 import { CoreLocalNotifications } from '@services/local-notifications';
 import { CoreEventObserver, CoreEvents } from '@singletons/events';
 import { CoreApp } from '@services/app';
-import { CoreSplitViewComponent } from '@components/split-view/split-view';
 import moment from 'moment';
 import { CoreConstants } from '@/core/constants';
 import { AddonCalendarFilterPopoverComponent } from '../../components/filter/filter';
@@ -51,7 +50,6 @@ import { CoreNavigator } from '@services/navigator';
 export class AddonCalendarListPage implements OnInit, OnDestroy {
 
     @ViewChild(IonContent) content?: IonContent;
-    @ViewChild(CoreSplitViewComponent) splitviewCtrl?: CoreSplitViewComponent;
 
     protected initialTime = 0;
     protected daysLoaded = 0;
@@ -117,26 +115,12 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
         this.newEventObserver = CoreEvents.on(AddonCalendarProvider.NEW_EVENT_EVENT, (data) => {
             if (data && data.eventId) {
                 this.eventsLoaded = false;
-                this.refreshEvents(true, false).finally(() => {
+                this.refreshEvents(true, false);
-
-                    // In tablet mode try to open the event (only if it's an online event).
-                    if (this.splitviewCtrl?.outletActivated && data.eventId > 0) {
-                        this.gotoEvent(data.eventId);
-                    } else if (this.splitviewCtrl?.outletActivated) {
-                        // Discussion added, clear details page.
-                        this.emptySplitView();
-                    }
-                });
             }
         }, this.currentSiteId);
 
         // Listen for new event discarded event. When it does, reload the data.
         this.discardedObserver = CoreEvents.on(AddonCalendarProvider.NEW_EVENT_DISCARDED_EVENT, () => {
-            if (this.splitviewCtrl?.outletActivated) {
-                // Discussion added, clear details page.
-                this.emptySplitView();
-            }
-
             this.eventsLoaded = false;
             this.refreshEvents(true, false);
         }, this.currentSiteId);
@@ -150,15 +134,9 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
         }, this.currentSiteId);
 
         // Refresh data if calendar events are synchronized automatically.
-        this.syncObserver = CoreEvents.on(AddonCalendarSyncProvider.AUTO_SYNCED, (data) => {
+        this.syncObserver = CoreEvents.on(AddonCalendarSyncProvider.AUTO_SYNCED, () => {
             this.eventsLoaded = false;
             this.refreshEvents();
-
-            if (this.splitviewCtrl?.outletActivated &&
-                this.eventId && data && data.deleted && data.deleted.indexOf(this.eventId) != -1) {
-                // Current selected event was deleted. Clear details.
-                this.emptySplitView();
-            }
         }, this.currentSiteId);
 
         // Refresh data if calendar events are synchronized manually but not by this page.
@@ -167,12 +145,6 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
                 this.eventsLoaded = false;
                 this.refreshEvents();
             }
-
-            if (this.splitviewCtrl?.outletActivated &&
-                this.eventId && data && data.deleted && data.deleted.indexOf(this.eventId) != -1) {
-                // Current selected event was deleted. Clear details.
-                this.emptySplitView();
-            }
         }, this.currentSiteId);
 
         // Update the list when an event is deleted.
@@ -185,11 +157,7 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
                     this.deletedEvents.push(data.eventId);
                     this.hasOffline = true;
                 } else {
-                    // Event deleted, clear the details if needed and refresh the view.
+                    // Event deleted, refresh the view.
-                    if (this.splitviewCtrl?.outletActivated) {
-                        this.emptySplitView();
-                    }
-
                     this.eventsLoaded = false;
                     this.refreshEvents();
                 }
@@ -248,27 +216,6 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
         this.syncIcon = CoreConstants.ICON_LOADING;
 
         await this.fetchData(false, true, false);
-
-        if (!this.eventId && this.splitviewCtrl?.outletActivated && this.events.length > 0) {
-            // Take first online event and load it. If no online event, load the first offline.
-            if (this.onlineEvents[0]) {
-                this.gotoEvent(this.onlineEvents[0].id);
-            } else {
-                this.gotoEvent(this.offlineEvents[0].id);
-            }
-        }
-    }
-
-    /**
-     * Convenience function to clear detail view of the split view.
-     */
-    protected emptySplitView(): void {
-        // Empty details.
-        const splitViewLoaded = CoreNavigator.isCurrentPathInTablet('**/calendar/list/event') ||
-            CoreNavigator.isCurrentPathInTablet('**/calendar/list/edit');
-        if (splitViewLoaded) {
-            CoreNavigator.navigate('../');
-        }
     }
 
     /**
@@ -633,20 +580,15 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
      */
     openEdit(eventId?: number): void {
         this.eventId = undefined;
+        eventId = eventId || 0;
 
         const params: Params = {};
 
-        if (eventId) {
-            params.eventId = eventId;
-        }
         if (this.filter.courseId) {
             params.courseId = this.filter.courseId;
         }
 
-        const splitViewLoaded = CoreNavigator.isCurrentPathInTablet('**/calendar/list/event') ||
+        CoreNavigator.navigateToSitePath(`calendar/edit/${eventId}`, { params });
-            CoreNavigator.isCurrentPathInTablet('**/calendar/list/edit');
-        const path = (splitViewLoaded ? '../' : '') + 'edit';
-        CoreNavigator.navigate(path, { params });
     }
 
     /**
@@ -664,16 +606,11 @@ export class AddonCalendarListPage implements OnInit, OnDestroy {
     gotoEvent(eventId: number): void {
         this.eventId = eventId;
 
-        if (eventId < 0) {
+        if (eventId <= 0) {
             // It's an offline event, go to the edit page.
             this.openEdit(eventId);
         } else {
-            const splitViewLoaded = CoreNavigator.isCurrentPathInTablet('**/calendar/list/event') ||
+            CoreNavigator.navigateToSitePath(`/calendar/event/${eventId}`);
-                CoreNavigator.isCurrentPathInTablet('**/calendar/list/edit');
-            const path = (splitViewLoaded ? '../' : '') + 'event';
-            CoreNavigator.navigate(path, { params: {
-                id: eventId,
-            } });
         }
     }
 

src/addons/mod/scorm/pages/player/player.ts

@@ -362,16 +362,6 @@ export class AddonModScormPlayerPage implements OnInit, OnDestroy {
         }
     }
 
-    /**
-     * Page will leave.
-     */
-    ionViewWillLeave(): void {
-        CoreEvents.trigger(CoreEvents.ACTIVITY_DATA_SENT, { module: 'scorm' });
-
-        // Empty src when leaving the state so unload event is triggered in the iframe.
-        this.src = '';
-    }
-
     /**
      * Load a SCO.
      *
@@ -547,6 +537,10 @@ export class AddonModScormPlayerPage implements OnInit, OnDestroy {
      * Component being destroyed.
      */
     ngOnDestroy(): void {
+        // Empty src when leaving the state so unload event is triggered in the iframe.
+        this.src = '';
+        CoreEvents.trigger(CoreEvents.ACTIVITY_DATA_SENT, { module: 'scorm' });
+
         // Stop listening for events.
         this.tocObserver?.off();
         this.launchNextObserver?.off();

src/core/directives/format-text.ts

@@ -377,6 +377,7 @@ export class CoreFormatTextDirective implements OnChanges {
 
             // Calculate the height now.
             this.calculateHeight();
+            setTimeout(() => this.calculateHeight(), 200); // Try again, sometimes the first calculation is wrong.
 
             // Add magnifying glasses to images.
             this.addMagnifyingGlasses();
@@ -388,6 +389,7 @@ export class CoreFormatTextDirective implements OnChanges {
                         if (data.loaded && CoreDomUtils.closest(this.element.parentElement, '#' + data.uniqueId)) {
                             // The format-text is inside the loading, re-calculate the height.
                             this.calculateHeight();
+                            setTimeout(() => this.calculateHeight(), 200);
                         }
                     });
             }

src/core/features/courses/courses.module.ts

@@ -38,11 +38,6 @@ export const CORE_COURSES_SERVICES: Type<unknown>[] = [
 ];
 
 const mainMenuHomeChildrenRoutes: Routes = [
-    {
-        path: '',
-        pathMatch: 'full',
-        redirectTo: CoreDashboardHomeHandlerService.PAGE_NAME,
-    },
     {
         path: CoreDashboardHomeHandlerService.PAGE_NAME,
         loadChildren: () => import('./pages/dashboard/dashboard.module').then(m => m.CoreCoursesDashboardPageModule),

src/core/features/siteplugins/services/siteplugins.ts

@@ -77,9 +77,6 @@ export class CoreSitePluginsProvider {
 
         const lang = await CoreLang.getCurrentLanguage();
 
-        // Clone the object so the original one isn't modified.
-        // const argsToSend = CoreUtils.clone(args);
-
         const defaultArgs: CoreSitePluginsDefaultArgs = {
             userid: <number> args.userid ?? site?.getUserId(),
             appid: CoreConstants.CONFIG.app_id,