EVOgeek/Vmeda.Online
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #2142 from dpalou/MOBILE-3199

Browse Source 
MOBILE-3199 lti: Launch LTI in browser in desktop
This commit is contained in:
Juan Leyva 2019-11-07 11:21:35 +01:00 committed by GitHub
commit dc35bc914f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/addon/mod/lti/providers/lti.ts
View File

@ -207,7 +207,12 @@ export class AddonModLtiProvider {
// Generate launcher and open it. // Generate launcher and open it.
return this.generateLauncher(url, params).then((url) => { return this.generateLauncher(url, params).then((url) => {
if (this.appProvider.isMobile()) {
this.utils.openInApp(url); this.utils.openInApp(url);
} else {
// In desktop open in browser, we found some cases where inapp caused JS issues.
this.utils.openInBrowser(url);
}
}); });
} }