jens/flipper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use isPWA utility function instead

Browse Source 
Summary: As a follow-up from the utility created on the previous diff

Reviewed By: aigoncharov

Differential Revision: D50885337

fbshipit-source-id: 6bc0c942b3d96eb020ec15395f34d5794ba2ae15
This commit is contained in:
Lorenzo Blasa
2023-11-01 11:15:17 -07:00
committed by Facebook GitHub Bot
parent 08e742ceff
commit 1a98038979
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
desktop/flipper-ui-browser/src/index.tsx
View File

@@ -15,6 +15,7 @@ import {
import {init as initLogger} from './fb-stubs/Logger'; import {init as initLogger} from './fb-stubs/Logger';
import {initializeRenderHost} from './initializeRenderHost'; import {initializeRenderHost} from './initializeRenderHost';
import {createFlipperServer, FlipperServerState} from 'flipper-server-client'; import {createFlipperServer, FlipperServerState} from 'flipper-server-client';
import {isPWA} from 'flipper-ui-core';
const loadingContainer = document.getElementById('loading'); const loadingContainer = document.getElementById('loading');
if (loadingContainer) { if (loadingContainer) {
@@ -151,7 +152,7 @@ start().catch((e) => {
logger.track('success-rate', 'flipper-ui-browser-started', { logger.track('success-rate', 'flipper-ui-browser-started', {
value: 0, value: 0,
error: getStringFromErrorLike(e), error: getStringFromErrorLike(e),
pwa: window.matchMedia('(display-mode: standalone)').matches, pwa: isPWA(),
}); });
window.flipperShowMessage?.('Failed to start UI with error: ' + e); window.flipperShowMessage?.('Failed to start UI with error: ' + e);
}); });

3
desktop/flipper-ui-core/src/chrome/PWAppInstallationWizard.tsx
View File

@@ -12,6 +12,7 @@ import {Image, Modal, Button} from 'antd';
import {getFlipperLib, Layout, _NuxManagerContext} from 'flipper-plugin'; import {getFlipperLib, Layout, _NuxManagerContext} from 'flipper-plugin';
import {getRenderHostInstance} from 'flipper-frontend-core'; import {getRenderHostInstance} from 'flipper-frontend-core';
import isProduction from '../utils/isProduction'; import isProduction from '../utils/isProduction';
import {isPWA} from '../utils/pwa';
type Props = { type Props = {
onHide: () => void; onHide: () => void;
@@ -55,7 +56,7 @@ export function shouldShowPWAInstallationWizard(): boolean {
return false; return false;
} }
if (window.matchMedia('(display-mode: standalone)').matches) { if (isPWA()) {
tracker.track('pwa-installation-wizard-should-show', { tracker.track('pwa-installation-wizard-should-show', {
show: false, show: false,
reason: 'Display mode is standalone, seems is already running as PWA', reason: 'Display mode is standalone, seems is already running as PWA',