1
0
mirror of synced 2025-12-21 02:46:50 -05:00

remove search popover (#56639)

This commit is contained in:
Evan Bonsignori
2025-07-15 13:05:52 -07:00
committed by GitHub
parent 4d38e797a3
commit 03e227c3b5

View File

@@ -4,10 +4,8 @@ import { CopilotIcon, SearchIcon } from '@primer/octicons-react'
import { useTranslation } from '@/languages/components/useTranslation' import { useTranslation } from '@/languages/components/useTranslation'
import { QueryParams } from '@/search/components/hooks/useMultiQueryParams' import { QueryParams } from '@/search/components/hooks/useMultiQueryParams'
import { useCTAPopoverContext } from '@/frame/components/context/CTAContext'
import styles from './SearchBarButton.module.scss' import styles from './SearchBarButton.module.scss'
import { AISearchCTAPopup } from './AISearchCTAPopup'
type Props = { type Props = {
isSearchOpen: boolean isSearchOpen: boolean
@@ -25,7 +23,6 @@ export function SearchBarButton({
instanceId, instanceId,
}: Props) { }: Props) {
const { t } = useTranslation('search') const { t } = useTranslation('search')
const { isOpen, dismiss } = useCTAPopoverContext()
const urlSearchInputQuery = params['search-overlay-input'] const urlSearchInputQuery = params['search-overlay-input']
@@ -57,15 +54,10 @@ export function SearchBarButton({
{/* We don't want to show the input when overlay is open */} {/* We don't want to show the input when overlay is open */}
{!isSearchOpen ? ( {!isSearchOpen ? (
<> <>
<AISearchCTAPopup
isOpen={isOpen}
setIsSearchOpen={setIsSearchOpen}
dismiss={dismiss}
instanceId={instanceId}
/>
{/* On mobile only the IconButton is shown */} {/* On mobile only the IconButton is shown */}
<IconButton <IconButton
data-testid="mobile-search-button" data-testid="mobile-search-button"
data-instance={instanceId}
ref={searchButtonRef} ref={searchButtonRef}
className={styles.searchIconButton} className={styles.searchIconButton}
onClick={handleClick} onClick={handleClick}
@@ -76,6 +68,7 @@ export function SearchBarButton({
{/* On large and up the SearchBarButton is shown */} {/* On large and up the SearchBarButton is shown */}
<button <button
data-testid="search" data-testid="search"
data-instance={instanceId}
tabIndex={0} tabIndex={0}
aria-label={t('search.input.placeholder_no_icon')} aria-label={t('search.input.placeholder_no_icon')}
className={styles.searchInputButton} className={styles.searchInputButton}