#id_dash_applet_menu now outside #id_applets_container to avoid clipping, other issues (FTs passed locally, but not in headless CI pipeline); selenium now calls wait_for when looking for is_displayed on kit bag menu (hopefully another CI fix)

src/functional_tests/test_game_kit.py

@@ -24,10 +24,11 @@ class GameKitTest(FunctionalTest):
         )
         self.assertTrue(kit_btn.is_displayed())
         kit_btn.click()
-        dialog = self.wait_for(
-            lambda: self.browser.find_element(By.ID, "id_kit_bag_dialog")
+        self.wait_for(
+            lambda: self.assertTrue(
+                self.browser.find_element(By.ID, "id_kit_bag_dialog").is_displayed()
+            )
         )
-        self.assertTrue(dialog.is_displayed())
 
     def test_kit_dialog_shows_token_cards(self):
         self.browser.get(self.gate_url)
@@ -42,10 +43,12 @@ class GameKitTest(FunctionalTest):
     def test_kit_dialog_closes_on_escape(self):
         self.browser.get(self.gate_url)
         self.browser.find_element(By.ID, "id_kit_btn").click()
-        dialog = self.wait_for(
-            lambda: self.browser.find_element(By.ID, "id_kit_bag_dialog")
+        self.wait_for(
+            lambda: self.assertTrue(
+                self.browser.find_element(By.ID, "id_kit_bag_dialog").is_displayed()
+            )
         )
-        self.assertTrue(dialog.is_displayed())
+        dialog = self.browser.find_element(By.ID, "id_kit_bag_dialog")
         dialog.send_keys(Keys.ESCAPE)
         self.wait_for(
             lambda: self.assertFalse(