From f405d6ef93406c0d227f449f52f58bf57d9b73ae Mon Sep 17 00:00:00 2001
From: vivekbopaliya <vivekpatel1nov@gmail.com>
Date: Sun, 10 Aug 2025 15:54:13 +0530
Subject: [PATCH] Changed scss for visibility dropdown

---
 client/modules/IDE/components/Header/MobileNav.jsx  | 13 -------------
 client/modules/IDE/components/Header/Toolbar.jsx    |  1 -
 client/modules/IDE/components/SketchListRowBase.jsx |  2 +-
 client/styles/components/_visibility-dropdown.scss  | 12 +++++++++---
 4 files changed, 10 insertions(+), 18 deletions(-)

diff --git a/client/modules/IDE/components/Header/MobileNav.jsx b/client/modules/IDE/components/Header/MobileNav.jsx
index 5a49691da7..c19d63ae40 100644
--- a/client/modules/IDE/components/Header/MobileNav.jsx
+++ b/client/modules/IDE/components/Header/MobileNav.jsx
@@ -249,8 +249,6 @@ const MobileNav = () => {
   const userIsOwner = user?.username === project.owner?.username;
   const Logo = AsteriskIcon;
 
-  const showPrivacyToggle =
-    project?.owner && title === project.name && userIsOwner;
   const showOwner = project?.owner && title === project.name && !userIsOwner;
 
   const toggleVisibility = (e) => {
@@ -274,17 +272,6 @@ const MobileNav = () => {
       </LogoContainer>
       <Title>
         <h1>{title === project?.name ? <ProjectName /> : title}</h1>
-        {showPrivacyToggle && (
-          <main className="toolbar__makeprivate">
-            <p>Private</p>
-            <input
-              className="toolbar__togglevisibility"
-              type="checkbox"
-              onChange={toggleVisibility}
-              defaultChecked={project.visibility === 'Private'}
-            />
-          </main>
-        )}
         {showOwner && <h5>by {project?.owner?.username}</h5>}
       </Title>
 
diff --git a/client/modules/IDE/components/Header/Toolbar.jsx b/client/modules/IDE/components/Header/Toolbar.jsx
index 0b7b061f0f..5227b74210 100644
--- a/client/modules/IDE/components/Header/Toolbar.jsx
+++ b/client/modules/IDE/components/Header/Toolbar.jsx
@@ -122,7 +122,6 @@ const Toolbar = (props) => {
           </div>
         )}
 
-        {/* ✅ Still show owner if not you */}
         {project?.owner && !userIsOwner && (
           <p className="toolbar__project-owner">
             {t('Toolbar.By')}{' '}
diff --git a/client/modules/IDE/components/SketchListRowBase.jsx b/client/modules/IDE/components/SketchListRowBase.jsx
index cfb85a52c6..b50cf63e4d 100644
--- a/client/modules/IDE/components/SketchListRowBase.jsx
+++ b/client/modules/IDE/components/SketchListRowBase.jsx
@@ -125,7 +125,7 @@ const SketchListRowBase = ({
       <th scope="row">{name}</th>
       <td>{formatDateCell(sketch.createdAt, mobile)}</td>
       <td>{formatDateCell(sketch.updatedAt, mobile)}</td>
-      <td hidden={!userIsOwner}>
+      <td hidden={!userIsOwner || mobile}>
         <VisibilityDropdown
           sketch={sketch}
           onVisibilityChange={handleVisibilityChange}
diff --git a/client/styles/components/_visibility-dropdown.scss b/client/styles/components/_visibility-dropdown.scss
index 9170922320..67479a487b 100644
--- a/client/styles/components/_visibility-dropdown.scss
+++ b/client/styles/components/_visibility-dropdown.scss
@@ -45,7 +45,7 @@
 
   @include themify() {
     border-bottom: 1px solid getThemifyVariable("modal-border-color");
-    background-color: white;
+    background-color: getThemifyVariable("modal-background-color");
   }
 
   &:last-child {
@@ -54,14 +54,20 @@
 
   &:hover {
     @include themify() {
-      background-color: getThemifyVariable("table-row-stripe-color") !important; 
+      background-color: getThemifyVariable("file-selected-color") !important; 
     }
   }
 
   &.selected {
     @include themify() {
-      background-color: getThemifyVariable("search-background-color");
+      background-color: getThemifyVariable("table-row-stripe-color");
     }
+
+    // &:hover {
+    //   @include themify() {
+    //     background-color: getThemifyVariable("table-row-stripe-color") !important;
+    //   }
+    // }
   }
 }