Skip to content

Commit dea38ff

Browse files
authored
[AI Search] minimum change to not persist search params (#55208)
1 parent 806775a commit dea38ff

File tree

3 files changed

+6
-14
lines changed

3 files changed

+6
-14
lines changed

src/fixtures/tests/playwright-rendering.spec.ts

+1-3
Original file line numberDiff line numberDiff line change
@@ -123,9 +123,7 @@ test('open new search, and select a general search article', async ({ page }) =>
123123
await page.keyboard.press('Enter')
124124

125125
// We should now be on the page for "For Playwright"
126-
await expect(page).toHaveURL(
127-
/\/get-started\/foo\/for-playwright\?search-overlay-input=serve\+playwright$/,
128-
)
126+
await expect(page).toHaveURL(/\/get-started\/foo\/for-playwright$/)
129127
await expect(page).toHaveTitle(/For Playwright/)
130128
})
131129

src/frame/components/ui/MarkdownContent/UnrenderedMarkdownContent.tsx

-6
Original file line numberDiff line numberDiff line change
@@ -44,12 +44,6 @@ export const UnrenderedMarkdownContent = ({
4444
if (existingURLParams.get('feature')) {
4545
newParams.set('feature', existingURLParams.get('feature') || '')
4646
}
47-
if (existingURLParams.get('search-overlay-ask-ai')) {
48-
newParams.set('search-overlay-ask-ai', existingURLParams.get('search-overlay-ask-ai') || '')
49-
}
50-
if (existingURLParams.get('search-overlay-input')) {
51-
newParams.set('search-overlay-input', existingURLParams.get('search-overlay-input') || '')
52-
}
5347
// Combine new and existing query parameters
5448
if (newParams.toString()) {
5549
href = `${href}?${existingAnchorParams}&${newParams.toString()}`

src/search/components/input/SearchOverlay.tsx

+5-5
Original file line numberDiff line numberDiff line change
@@ -332,6 +332,9 @@ export function SearchOverlay({
332332
if (searchParams.has('search-overlay-open')) {
333333
searchParams.delete('search-overlay-open')
334334
}
335+
if (searchParams.has('search-overlay-input')) {
336+
searchParams.delete('search-overlay-input')
337+
}
335338
router.push(`${selectedOption.url}?${searchParams.toString()}` || '')
336339
onClose()
337340
}
@@ -373,11 +376,8 @@ export function SearchOverlay({
373376
eventGroupId: askAIEventGroupId.current,
374377
})
375378
setSelectedIndex(-1)
376-
const searchParams = new URLSearchParams((router.query as Record<string, string>) || {})
377-
if (searchParams.has('search-overlay-open')) {
378-
searchParams.delete('search-overlay-open')
379-
}
380-
window.open(`${url}?${searchParams.toString()}` || '', '_blank')
379+
// hardcode the feature to ai_search for now, delete when GA is ready
380+
window.open(`${url}?feature=ai_search` || '', '_blank')
381381
}
382382

383383
// Handle keyboard navigation of suggestions

0 commit comments

Comments
 (0)