From 9d310b3a4d4a3fb119993e65f07ac5e80dde662a Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles@charleskerr.com>
Date: Wed, 22 Feb 2023 18:31:24 -0600
Subject: [PATCH] fix: padding in overflow menu (#5001)

---
 web/assets/css/transmission-app.scss | 14 +++++++-------
 web/src/overflow-menu.js             |  3 ++-
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/web/assets/css/transmission-app.scss b/web/assets/css/transmission-app.scss
index f6a31a7fa..fe1852b25 100644
--- a/web/assets/css/transmission-app.scss
+++ b/web/assets/css/transmission-app.scss
@@ -1282,13 +1282,6 @@ $video-image: '../img/film.svg';
   @include for-phone-only() {
     height: 80vh;
   }
-  // hide the fullscreen button unless we're on mobile
-  @include for-tablet-portrait-up {
-    #display-fullscreen-check,
-    #display-fullscreen-label {
-      display: none;
-    }
-  }
 
   fieldset {
     border: 0;
@@ -1315,6 +1308,13 @@ $video-image: '../img/film.svg';
     display: flex;
     flex-direction: row;
     padding: 3px 0;
+
+    // hide the fullscreen button unless we're on mobile
+    @include for-tablet-portrait-up {
+      &.display-fullscreen-row {
+        display: none;
+      }
+    }
   }
 
   legend {
diff --git a/web/src/overflow-menu.js b/web/src/overflow-menu.js
index 30ec94904..01387ec52 100644
--- a/web/src/overflow-menu.js
+++ b/web/src/overflow-menu.js
@@ -248,7 +248,8 @@ export class OverflowMenu extends EventTarget {
     // fullscreen
 
     div = document.createElement('div');
-    div.classList.add('table-row');
+    div.classList.add('table-row', 'display-fullscreen-row');
+
     options.append(div);
 
     check = document.createElement('input');