From d198f1947182e4979b6fd4a1deb341f6f7d64f79 Mon Sep 17 00:00:00 2001 From: Anton Kastritskiy Date: Tue, 18 Jul 2023 03:52:34 -0700 Subject: [PATCH] rearrange app inspect section Summary: - remove tooltip as it is about the items that are in the navbar now - metro buttons and bookmarks are still located in the sidebar - so are plugin app actions Reviewed By: elboman Differential Revision: D47470797 fbshipit-source-id: cb7d757821eaa45ea770d3731b94e4e2f1a6d0ff --- .../sandy-chrome/appinspect/AppInspect.tsx | 23 ++++++------------- 1 file changed, 7 insertions(+), 16 deletions(-) diff --git a/desktop/flipper-ui-core/src/sandy-chrome/appinspect/AppInspect.tsx b/desktop/flipper-ui-core/src/sandy-chrome/appinspect/AppInspect.tsx index 686cdc06d..72dcbb6dd 100644 --- a/desktop/flipper-ui-core/src/sandy-chrome/appinspect/AppInspect.tsx +++ b/desktop/flipper-ui-core/src/sandy-chrome/appinspect/AppInspect.tsx @@ -9,8 +9,8 @@ import React from 'react'; import {Typography} from 'antd'; -import {LeftSidebar, SidebarTitle, InfoIcon} from '../LeftSidebar'; -import {Layout, Link, styled} from '../../ui'; +import {LeftSidebar, SidebarTitle} from '../LeftSidebar'; +import {Layout, styled} from '../../ui'; import {theme, useValue} from 'flipper-plugin'; import {PluginList} from './PluginList'; import MetroButton from '../../chrome/MetroButton'; @@ -29,15 +29,6 @@ import {PluginActionsMenu} from '../../chrome/PluginActionsMenu'; const {Text} = Typography; -const appTooltip = ( - <> - Inspect apps by selecting connected devices and emulators. Navigate and - bookmark frequent destinations in the app. Refresh, screenshot and - screenrecord is also available.{' '} - Learn More - -); - export function AppInspect() { const metroDevice = useSelector(getMetroDevice); const client = useSelector(getActiveClient); @@ -49,16 +40,16 @@ export function AppInspect() { - {appTooltip}}> - App Inspect - + + App Inspect +
+ + {isDeviceConnected && isAppConnected && } {isDeviceConnected && activeDevice && ( -
- )}