From c901801c6577a1e5eb8be6a51edfe8cf89dd0fe4 Mon Sep 17 00:00:00 2001
From: Akita Noek <anoek@online-go.com>
Date: Fri, 20 Dec 2024 05:51:18 -0700
Subject: [PATCH] Fix rank selection menu placement on mobile

---
 src/views/Play/QuickMatch.styl |  8 ++++++
 src/views/Play/QuickMatch.tsx  | 46 +++++++---------------------------
 2 files changed, 17 insertions(+), 37 deletions(-)

diff --git a/src/views/Play/QuickMatch.styl b/src/views/Play/QuickMatch.styl
index 1cc0f311a8..66693ec140 100644
--- a/src/views/Play/QuickMatch.styl
+++ b/src/views/Play/QuickMatch.styl
@@ -474,6 +474,14 @@
         align-items: center;
         gap: 0.5rem;
     }
+
+    .opponent-rank-container {
+        .ogs-react-select__menu {
+            right: 0;
+            width: 6rem;
+            max-width: 40vw;
+        }
+    }
 }
 
 .PlayButton-container {
diff --git a/src/views/Play/QuickMatch.tsx b/src/views/Play/QuickMatch.tsx
index 570051fec5..aaaa928bcb 100644
--- a/src/views/Play/QuickMatch.tsx
+++ b/src/views/Play/QuickMatch.tsx
@@ -927,7 +927,7 @@ export function QuickMatch(): JSX.Element {
 
                 {/* Play Button */}
                 <div className="GameOption-cell">
-                    <div className="GameOption">
+                    <div className="GameOption opponent-rank-container">
                         <span>{_("Opponent rank")}</span>
                         <div className="opponent-rank-range">
                             <Select
@@ -935,8 +935,15 @@ export function QuickMatch(): JSX.Element {
                                 value={lower_rank_diff_options.find(
                                     (o) => o.value === lower_rank_diff.toString(),
                                 )}
+                                styles={{
+                                    menu: (base) => ({
+                                        ...base,
+                                        width: "120",
+                                    }),
+                                }}
                                 isSearchable={false}
                                 isDisabled={automatch_search_active}
+                                menuPlacement="auto"
                                 onChange={(opt) => {
                                     if (opt) {
                                         setLowerRankDiff(parseInt(opt.value));
@@ -952,24 +959,6 @@ export function QuickMatch(): JSX.Element {
                                 }}
                             />
 
-                            {/*
-                            <select
-                                value={lower_rank_diff}
-                                onChange={(ev) => setLowerRankDiff(parseInt(ev.target.value))}
-                                disabled={automatch_search_active}
-                            >
-                                {user.anonymous ? (
-                                    <option>{"30k"}</option>
-                                ) : (
-                                    [9, 8, 7, 6, 5, 4, 3, 2, 1, 0].map((v) => (
-                                        <option key={v} value={v}>
-                                            - {v}
-                                        </option>
-                                    ))
-                                )}
-                            </select>
-                            */}
-
                             {" - "}
 
                             <Select
@@ -979,6 +968,7 @@ export function QuickMatch(): JSX.Element {
                                 )}
                                 isSearchable={false}
                                 isDisabled={automatch_search_active}
+                                menuPlacement="auto"
                                 onChange={(opt) => {
                                     if (opt) {
                                         setUpperRankDiff(parseInt(opt.value));
@@ -993,24 +983,6 @@ export function QuickMatch(): JSX.Element {
                                     Option: RenderOptionWithDescription,
                                 }}
                             />
-
-                            {/*
-                            <select
-                                value={upper_rank_diff}
-                                onChange={(ev) => setUpperRankDiff(parseInt(ev.target.value))}
-                                disabled={automatch_search_active}
-                            >
-                                {user.anonymous ? (
-                                    <option>{"9d"}</option>
-                                ) : (
-                                    [0, 1, 2, 3, 4, 5, 6, 7, 8, 9].map((v) => (
-                                        <option key={v} value={v}>
-                                            + {v}
-                                        </option>
-                                    ))
-                                )}
-                            </select>
-                            */}
                         </div>
                     </div>
                 </div>