From 124deda6a460a833d7e86a2465e6290bba920f4c Mon Sep 17 00:00:00 2001
From: tabarra <1808295+tabarra@users.noreply.github.com>
Date: Fri, 19 Apr 2024 09:06:50 -0300
Subject: [PATCH] fix(web): small css issues from v7.1

---
 docs/dev_notes.md                                | 4 ++--
 panel/src/layout/ActionModal/ActionIdsTab.tsx    | 2 +-
 panel/src/layout/ActionModal/ActionInfoTab.tsx   | 2 +-
 panel/src/layout/ActionModal/ActionModifyTab.tsx | 2 +-
 panel/src/layout/PlayerModal/PlayerBanTab.tsx    | 2 +-
 panel/src/pages/Players/PlayersSearchBox.tsx     | 4 ++--
 6 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/docs/dev_notes.md b/docs/dev_notes.md
index 55e708f3e..4a87a4e59 100644
--- a/docs/dev_notes.md
+++ b/docs/dev_notes.md
@@ -7,10 +7,10 @@
 
 ## v7.1.0 bugs
 - [x] unrenderable legacy 404 page
-- [ ] "Search by Player IDs" filter on players page is overflowing
+- [x] "Search by Player IDs" filter on players page is overflowing
+- [x] history action modal needs py-1 because of the reason field outline
 - [ ] ctrl+f doesn't work in the player modal anymore, if on the player or history pages
     - criar um estado "any modal open" pra desabilitar todos hotkeys das páginas?
-- [ ] history action modal needs py-1 because of the reason field outline
 - [ ] make the new page searches case insensitive (history->reason and maybe more)
 - [ ] remove all "blur" as that is slow as hell for browsers with hw acceleration disabled
 
diff --git a/panel/src/layout/ActionModal/ActionIdsTab.tsx b/panel/src/layout/ActionModal/ActionIdsTab.tsx
index 1bd05a202..78d0e4672 100644
--- a/panel/src/layout/ActionModal/ActionIdsTab.tsx
+++ b/panel/src/layout/ActionModal/ActionIdsTab.tsx
@@ -31,7 +31,7 @@ function IdsBlock({ title, emptyMessage, ids, isSmaller }: IdsBlockProps) {
     }
     const hasIdsAvailable = Array.isArray(ids) && ids.length;
 
-    return <div className="mb-1 md:mb-4">
+    return <div className="px-1 mb-1 md:mb-4">
         <div className="flex justify-between items-center pb-1">
             <h3 className="text-xl">{title}</h3>
             {/* {hasCopiedIds ? (
diff --git a/panel/src/layout/ActionModal/ActionInfoTab.tsx b/panel/src/layout/ActionModal/ActionInfoTab.tsx
index 5985dbea8..04b89d294 100644
--- a/panel/src/layout/ActionModal/ActionInfoTab.tsx
+++ b/panel/src/layout/ActionModal/ActionInfoTab.tsx
@@ -91,7 +91,7 @@ export default function ActionInfoTab({ action, serverTime, tsFetch }: ActionInf
         openPlayerModal({ license: linkedPlayer });
     }
 
-    return <div className="mb-1 md:mb-4">
+    return <div className="px-1 mb-1 md:mb-4">
         <dl className="pb-2">
             <div className="py-0.5 grid grid-cols-3 gap-4 px-0">
                 <dt className="text-sm font-medium leading-6 text-muted-foreground">Date/Time</dt>
diff --git a/panel/src/layout/ActionModal/ActionModifyTab.tsx b/panel/src/layout/ActionModal/ActionModifyTab.tsx
index c6f0343f1..1f7dbcdc5 100644
--- a/panel/src/layout/ActionModal/ActionModifyTab.tsx
+++ b/panel/src/layout/ActionModal/ActionModifyTab.tsx
@@ -46,7 +46,7 @@ export default function ActionModifyTab({ action, refreshModalData }: ActionModi
             ? `Revoke ${upperCasedType}`
             : 'Revoke (no permission)';
     return (
-        <div className="flex flex-col gap-4 mb-1 md:mb-4">
+        <div className="flex flex-col gap-4 px-1 mb-1 md:mb-4">
             <div className="space-y-2">
                 <h3 className="text-xl">Revoke {upperCasedType}</h3>
                 <p className="text-muted-foreground text-sm">
diff --git a/panel/src/layout/PlayerModal/PlayerBanTab.tsx b/panel/src/layout/PlayerModal/PlayerBanTab.tsx
index 7cb03d9fa..4521c76d3 100644
--- a/panel/src/layout/PlayerModal/PlayerBanTab.tsx
+++ b/panel/src/layout/PlayerModal/PlayerBanTab.tsx
@@ -73,7 +73,7 @@ export default function PlayerBanTab({ playerRef }: { playerRef: PlayerModalRefT
                 <Label htmlFor="durationSelect" className="col-span-4 xs:col-auto">
                     Duration
                 </Label>
-                <div className="col-span-full xs:col-span-3 space-y-1 ">
+                <div className="col-span-full xs:col-span-3 space-y-1">
                     <Select
                         onValueChange={setCurrentDuration}
                         defaultValue={currentDuration}
diff --git a/panel/src/pages/Players/PlayersSearchBox.tsx b/panel/src/pages/Players/PlayersSearchBox.tsx
index 84c7ac50d..041e9a019 100644
--- a/panel/src/pages/Players/PlayersSearchBox.tsx
+++ b/panel/src/pages/Players/PlayersSearchBox.tsx
@@ -175,13 +175,13 @@ export function PlayerSearchBox({ doSearch, initialState }: PlayerSearchBoxProps
                                 role="combobox"
                                 aria-expanded={isSearchTypeDropdownOpen}
                                 onClick={() => setSearchTypeDropdownOpen(!isSearchTypeDropdownOpen)}
-                                className="xs:w-40 justify-between border-input bg-black/5 dark:bg-black/30 hover:dark:bg-primary grow md:grow-0"
+                                className="xs:w-48 justify-between border-input bg-black/5 dark:bg-black/30 hover:dark:bg-primary grow md:grow-0"
                             >
                                 Search by {selectedSearchType.label}
                                 <ChevronsUpDownIcon className="ml-2 h-4 w-4 shrink-0 opacity-50" />
                             </Button>
                         </DropdownMenuTrigger>
-                        <DropdownMenuContent className='w-40'>
+                        <DropdownMenuContent className='w-48'>
                             <DropdownMenuLabel>Search Type</DropdownMenuLabel>
                             <DropdownMenuSeparator />
                             <DropdownMenuRadioGroup value={currSearchType} onValueChange={setCurrSearchType}>