fix: add keyboard shortcut to inside button on search result preview (#…

…27071) Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
PostHog · Dec 19, 2024 · ffe4e9d · ffe4e9d
1 parent 3843cc0
commit ffe4e9d
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 14 deletions.
diff --git a/frontend/src/lib/components/CommandBar/SearchResultPreview.tsx b/frontend/src/lib/components/CommandBar/SearchResultPreview.tsx
@@ -40,18 +40,10 @@ export const SearchResultPreview = (): JSX.Element | null => {
                         onClick={() => {
                             openResult(activeResultIndex)
                         }}
-                        tooltip={
-                            <>
-                                Open <KeyboardShortcut enter />
-                            </>
-                        }
                         aria-label="Open search result"
                     >
-                        Open
+                        <span className="mr-1">Open</span> <KeyboardShortcut enter />
                     </LemonButton>
-                    <div>
-                        <KeyboardShortcut enter /> Open
-                    </div>
                 </div>
             </div>
         </div>

diff --git a/frontend/src/lib/lemon-ui/LemonInput/LemonInput.scss b/frontend/src/lib/lemon-ui/LemonInput/LemonInput.scss
@@ -98,11 +98,6 @@
         // NOTE Design: Search inputs are given a specific small width
         max-width: 240px;
         border-radius: 0;
-
-        // Add a focus ring to the element (not the input) when focused
-        &:has(:focus-visible) {
-            outline: -webkit-focus-ring-color auto 1px;
-        }
     }
 
     &.LemonInput--type-number {