Skip to content

Repo sync #39338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jul 15, 2025
Merged

Repo sync #39338

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
dcf6e17
Update OpenAPI Description (#56649)
docs-bot Jul 15, 2025
89df927
Update audit log event data (#56648)
docs-bot Jul 15, 2025
8a116f7
Updating Visual Studio docs on agent mode request counting (#56610)
andysterland Jul 15, 2025
d0fe048
EDI-ify "About workflows" (#56631)
sabrowning1 Jul 15, 2025
4442df8
GraphQL schema update (#56651)
docs-bot Jul 15, 2025
4d38e79
Display event fields on audit log pages (#56332)
heiskr Jul 15, 2025
03e227c
remove search popover (#56639)
Ebonsignori Jul 15, 2025
52464b7
Correct initial paragraph for configuring SAML SSO for EMUs (#56644)
lildude Jul 15, 2025
370b4e4
Patch release notes for GitHub Enterprise Server (#56608)
release-controller[bot] Jul 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove search popover (#56639)
  • Loading branch information
@Ebonsignori
Ebonsignori authored Jul 15, 2025
commit 03e227c3b5655e23b1b2389e1efb9a5ff33596d7
11 changes: 2 additions & 9 deletions src/search/components/input/SearchBarButton.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,8 @@ import { CopilotIcon, SearchIcon } from '@primer/octicons-react'

import { useTranslation } from '@/languages/components/useTranslation'
import { QueryParams } from '@/search/components/hooks/useMultiQueryParams'
import { useCTAPopoverContext } from '@/frame/components/context/CTAContext'

import styles from './SearchBarButton.module.scss'
import { AISearchCTAPopup } from './AISearchCTAPopup'

type Props = {
isSearchOpen: boolean
Expand All @@ -25,7 +23,6 @@ export function SearchBarButton({
instanceId,
}: Props) {
const { t } = useTranslation('search')
const { isOpen, dismiss } = useCTAPopoverContext()

const urlSearchInputQuery = params['search-overlay-input']

Expand Down Expand Up @@ -57,15 +54,10 @@ export function SearchBarButton({
{/* We don't want to show the input when overlay is open */}
{!isSearchOpen ? (
<>
<AISearchCTAPopup
isOpen={isOpen}
setIsSearchOpen={setIsSearchOpen}
dismiss={dismiss}
instanceId={instanceId}
/>
{/* On mobile only the IconButton is shown */}
<IconButton
data-testid="mobile-search-button"
data-instance={instanceId}
ref={searchButtonRef}
className={styles.searchIconButton}
onClick={handleClick}
Expand All @@ -76,6 +68,7 @@ export function SearchBarButton({
{/* On large and up the SearchBarButton is shown */}
<button
data-testid="search"
data-instance={instanceId}
tabIndex={0}
aria-label={t('search.input.placeholder_no_icon')}
className={styles.searchInputButton}
Expand Down