Fix UI Tweaks sidebar project row overflow (ilysenko#1129)

* Cover sidebar project row geometry

* Avoid sidebar project row overflow

Author: PinguuSS

linux-features/ui-tweaks/README.md

@@ -170,7 +170,7 @@ Tracked default in `feature.json`:
     "sidebar": {
       "projectName": {
         "enabled": true,
-        "style": "font-weight: 700 !important; padding-top: 0.25rem;"
+        "style": "font-weight: 700 !important;"
       }
     }
   }
@@ -184,8 +184,8 @@ Config keys:
 - `style`: CSS declaration list inserted into the project-name rule, such as
   `font-weight: 800 !important; color: red;`. It is not arbitrary CSS; unsafe
   syntax that could escape the scoped rule warns and falls back to the default.
-  The default is `font-weight: 700 !important; padding-top: 0.25rem;`, so
-  project names are bold with a small top offset and no color is forced.
+  The default is `font-weight: 700 !important;`, so project names are bold
+  without changing the fixed row geometry or forcing a color.
 
 ## Drift Behavior
 

linux-features/ui-tweaks/feature.json

@@ -39,7 +39,7 @@
     "sidebar": {
       "projectName": {
         "enabled": true,
-        "style": "font-weight: 700 !important; padding-top: 0.25rem;"
+        "style": "font-weight: 700 !important;"
       }
     }
   }

linux-features/ui-tweaks/patches/sidebar-project-name.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const DEFAULT_PROJECT_NAME_STYLE = "font-weight: 700 !important; padding-top: 0.25rem;";
+const DEFAULT_PROJECT_NAME_STYLE = "font-weight: 700 !important;";
 const PROJECTS_SIDEBAR_ASSET_PATTERN =
   /^app-initial~notebook-preview-panel~app-main~pull-request-route~projects-index-page~cloud-en~lpx9dmpy-[^.]+\.js$/;
 const PROJECT_NAME_SELECTOR = ".group\\/folder-row .text-fade-truncate.pr-1";

linux-features/ui-tweaks/test.js

@@ -504,17 +504,18 @@ test("feature manifest defaults reach descriptor context through the feature loa
     const [descriptor] = loadLinuxFeaturePatchDescriptors({ featuresRoot });
     const patched = descriptor.apply(projectBundleFixture(), {});
 
-    assert.match(patched, /font-weight: 700 !important; padding-top: 0.25rem;/);
+    assert.match(patched, /font-weight: 700 !important;/);
+    assert.doesNotMatch(patched, /padding-top/);
   } finally {
     fs.rmSync(tempDir, { recursive: true, force: true });
   }
 });
 
-test("default project name style is bold with top padding and no forced color", () => {
+test("default project name style is bold without changing fixed row geometry", () => {
   const featureJson = JSON.parse(fs.readFileSync(path.join(__dirname, "feature.json"), "utf8"));
   assert.equal(featureJson.tweaks.sidebar.projectName.style, DEFAULT_PROJECT_NAME_STYLE);
   assert.match(DEFAULT_PROJECT_NAME_STYLE, /font-weight:\s*700\s*!important/);
-  assert.match(DEFAULT_PROJECT_NAME_STYLE, /padding-top:\s*0\.25rem/);
+  assert.doesNotMatch(DEFAULT_PROJECT_NAME_STYLE, /(?:padding|margin|height)/i);
   assert.doesNotMatch(DEFAULT_PROJECT_NAME_STYLE, /color/i);
   assert.doesNotMatch(sidebarProjectNameCss(DEFAULT_PROJECT_NAME_STYLE), /#000|black/i);
 });
@@ -573,7 +574,8 @@ test("invalid feature settings warn and fall back to defaults", () => {
     const patched = descriptors[0].apply(projectBundleFixture(), {});
 
     assert.match(warnings.join("\n"), /WARN: Linux feature 'ui-tweaks' settings/);
-    assert.match(patched, /font-weight: 700 !important; padding-top: 0.25rem;/);
+    assert.match(patched, /font-weight: 700 !important;/);
+    assert.doesNotMatch(patched, /padding-top/);
   } finally {
     fs.rmSync(tempDir, { recursive: true, force: true });
   }
@@ -638,7 +640,8 @@ test("invalid and empty styles warn and fall back without throwing", () => {
     );
 
     assert.match(value, new RegExp(STYLE_ID));
-    assert.match(value, /font-weight: 700 !important; padding-top: 0.25rem;/);
+    assert.match(value, /font-weight: 700 !important;/);
+    assert.doesNotMatch(value, /padding-top/);
     assert.equal(warnings.length, 1);
     assert.match(warnings[0], /^WARN: ui-tweaks sidebar project name style/);
   }
@@ -663,7 +666,8 @@ test("unsafe styles warn, stay scoped, and fall back to the default", () => {
   );
 
   assert.match(value, new RegExp(STYLE_ID));
-  assert.match(value, /font-weight: 700 !important; padding-top: 0.25rem;/);
+  assert.match(value, /font-weight: 700 !important;/);
+  assert.doesNotMatch(value, /padding-top/);
   assert.doesNotMatch(value, /body\{display:none\}/);
   assert.equal(value.includes(unsafeStyle), false);
   assert.equal(warnings.length, 1);