Merge pull request #3588 from processing/feat/privatesketch

Feat/privatesketch

Author: raclim

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>
 

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')}{' '}

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}

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;
+    //   }
+    // }
   }
 }