javascript/ionic-framework
1
0
Fork 0
mirror of https://github.com/ionic-team/ionic-framework.git synced 2025-08-15 17:42:15 +08:00
Code Issues Packages Projects Releases Wiki Activity

feat(platform): add ability to override platform detection methods (#23915)

Browse Source 
resolves #19737
This commit is contained in:
William Martin
2021-09-14 15:49:04 -04:00
committed by GitHub
parent 414f24685c
commit 45cabae04b
8 changed files with 45 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
angular/src/index.ts
View File

@ -45,7 +45,7 @@ export * from './types/ionic-lifecycle-hooks';
export { IonicModule } from './ionic-module';
// UTILS
export { IonicSafeString, getPlatforms, isPlatform, createAnimation, IonicSwiper } from '@ionic/core';
export { PlatformConfig, IonicSafeString, getPlatforms, isPlatform, createAnimation, IonicSwiper } from '@ionic/core';
// CORE TYPES
export {

6
core/src/global/ionic-global.ts
View File

@ -21,9 +21,6 @@ export const initialize = (userConfig: IonicConfig = {}) => {
Context.config = config;
const Ionic = (win as any).Ionic = (win as any).Ionic || {};
// Setup platforms
setupPlatforms(win);
const platformHelpers: any = {};
if (userConfig._ael) {
platformHelpers.ael = userConfig._ael;
@ -51,6 +48,9 @@ export const initialize = (userConfig: IonicConfig = {}) => {
saveConfig(win, configObj);
}
// Setup platforms
setupPlatforms(win);
// first see if the mode was set as an attribute on <html>
// which could have been set by the user, or by pre-rendering
// otherwise get the mode via config settings, and fallback to md

2
core/src/index.ts
View File

@ -7,7 +7,7 @@ export { getTimeGivenProgression } from './utils/animation/cubic-bezier';
export { createGesture } from './utils/gesture';
export { initialize } from './global/ionic-global';
export { componentOnReady } from './utils/helpers';
export { isPlatform, Platforms, getPlatforms } from './utils/platform';
export { isPlatform, Platforms, PlatformConfig, getPlatforms } from './utils/platform';
export { IonicSafeString } from './utils/sanitization';
export { IonicConfig, getMode, setupConfig } from './utils/config';
export { LIFECYCLE_WILL_ENTER, LIFECYCLE_DID_ENTER, LIFECYCLE_WILL_LEAVE, LIFECYCLE_DID_LEAVE, LIFECYCLE_WILL_UNLOAD } from './components/nav/constants';

7
core/src/utils/config.ts
View File

@ -1,5 +1,7 @@
import { AnimationBuilder, Mode, SpinnerTypes, TabButtonLayout } from '../interface';
import { PlatformConfig } from './platform';
export interface IonicConfig {
/**
* When it's set to `false`, disables all animation and transition across the app.
@ -175,6 +177,11 @@ export interface IonicConfig {
*/
sanitizerEnabled?: boolean;
/**
* Overrides the default platform detection methods.
*/
platform?: PlatformConfig;
// PRIVATE configs
keyboardHeight?: number;
inputShims?: boolean;

12
core/src/utils/platform.ts
View File

@ -1,3 +1,4 @@
import { config } from '../global/config';
export type Platforms = keyof typeof PLATFORMS_MAP;
@ -29,8 +30,13 @@ export const setupPlatforms = (win: any = window) => {
return platforms;
};
const detectPlatforms = (win: Window) =>
(Object.keys(PLATFORMS_MAP) as Platforms[]).filter(p => PLATFORMS_MAP[p](win));
const detectPlatforms = (win: Window) => {
const customPlatformMethods = config.get('platform');
return (Object.keys(PLATFORMS_MAP) as Platforms[]).filter(p => {
const customMethod = customPlatformMethods && customPlatformMethods[p];
return typeof customMethod === 'function' ? customMethod(win) : PLATFORMS_MAP[p](win);
});
}
const isMobileWeb = (win: Window): boolean =>
isMobile(win) && !isHybrid(win);
@ -117,6 +123,8 @@ export const testUserAgent = (win: Window, expr: RegExp) =>
const matchMedia = (win: Window, query: string): boolean =>
win.matchMedia && win.matchMedia(query).matches;
export type PlatformConfig = Partial<typeof PLATFORMS_MAP>;
const PLATFORMS_MAP = {
'ipad': isIpad,
'iphone': isIphone,

19
core/src/utils/test/platform.spec.ts
View File

@ -1,3 +1,5 @@
import initialize from '../../global/ionic-global';
import { testUserAgent, getPlatforms, isPlatform } from '../platform';
import { PlatformConfiguration, configureBrowser } from './platform.utils';
@ -132,4 +134,21 @@ describe('Platform Tests', () => {
expect(isPlatform(win, 'android')).toEqual(false);
});
})
describe('Custom Platform Config', () => {
it('should use custom platform detection methods', () => {
const win = configureBrowser(PlatformConfiguration.DesktopSafari);
initialize({
'platform': {
'desktop': (win) => false,
'cordova': (win) => true
}
});
expect(isPlatform(win, 'desktop')).toEqual(false);
expect(isPlatform(win, 'cordova')).toEqual(true);
expect(getPlatforms(win).includes('cordova')).toEqual(true);
});
})
});

1
packages/react/src/components/index.ts
View File

@ -36,6 +36,7 @@ export {
setupConfig,
initialize,
IonicSwiper,
PlatformConfig,
SpinnerTypes,

2
packages/vue/src/index.ts
View File

@ -41,7 +41,7 @@ export {
IonicSafeString,
// Platform
isPlatform, Platforms, getPlatforms,
isPlatform, Platforms, PlatformConfig, getPlatforms,
// Gesture
Gesture,