remove hide/show rating button from settings

zubyj · zubyj · commit 4c63eca4bbc9 · 2025-04-12T18:17:47.000-07:00
diff --git a/src/background/background.ts b/src/background/background.ts
@@ -40,10 +40,9 @@ chrome.runtime.onInstalled.addListener(() => {
         });
 
     // Load default settings
-    chrome.storage.local.set({ fontSize: 12 }); // Default to small display size
+    chrome.storage.local.set({ fontSize: 14 });
     chrome.storage.local.set({ showExamples: true });
     chrome.storage.local.set({ showDifficulty: true });
-    chrome.storage.local.set({ showRating: true });
     chrome.storage.local.set({ showCompanyTags: true });
     // Set default theme to auto mode and default to dark
     chrome.storage.local.set({ isDarkTheme: true }); // Default to dark theme
diff --git a/src/content-script/update-description-tab.ts b/src/content-script/update-description-tab.ts
@@ -105,57 +105,46 @@ function showDifficulty() {
     });
 }
 
-// show the leetcode problem rating if the user has enabled it in the settings
+// show the leetcode problem rating
 function showRating(problemTitle: string) {
     // Check if we're on the description tab before proceeding
     const isDescriptionPage = !window.location.href.includes('/solutions');
     if (!isDescriptionPage) {
         return;
     }
 
-    chrome.storage.local.get(['showRating'], (result) => {
-        const showRating = result.showRating;
-        if (!showRating) {
-            const ratingElement = document.getElementById('rating');
-            if (ratingElement) {
-                ratingElement.remove();
-            }
-            return;
-        }
-
-        chrome.storage.local.get(['leetcodeProblems'], (result) => {
-            const problem = result.leetcodeProblems.questions.find((problem: Problem) => problem.title === problemTitle);
-            if (!problem?.rating) return;
-
-            let ratingElement = document.getElementById('rating');
-            if (!ratingElement) {
-                ratingElement = document.createElement('div');
-                ratingElement.id = 'rating';
-            }
-
-            ratingElement.textContent = problem.rating;
-            ratingElement.style.fontSize = '11px';
-            ratingElement.style.letterSpacing = '.5px';
-            ratingElement.style.borderRadius = '6px';
-            ratingElement.style.width = '60px';
-            ratingElement.style.textAlign = 'center';
-            ratingElement.style.padding = '4px 8px';
-            ratingElement.style.transition = 'all 0.2s ease';
+    chrome.storage.local.get(['leetcodeProblems'], (result) => {
+        const problem = result.leetcodeProblems.questions.find((problem: Problem) => problem.title === problemTitle);
+        if (!problem?.rating) return;
 
-            chrome.storage.local.get(['isDarkTheme'], (result) => {
-                const isDark = result.isDarkTheme;
-                if (ratingElement) {
-                    ratingElement.style.backgroundColor = isDark ? '#373737' : '#f3f4f5';
-                    ratingElement.style.color = isDark ? '#40a9ff' : '#1a1a1a';
-                    ratingElement.style.border = `1px solid ${isDark ? 'rgba(255, 255, 255, 0.1)' : 'rgba(0, 0, 0, 0.1)'}`;
-                }
-            });
+        let ratingElement = document.getElementById('rating');
+        if (!ratingElement) {
+            ratingElement = document.createElement('div');
+            ratingElement.id = 'rating';
+        }
 
-            const difficultyContainer = document.querySelectorAll('div.relative.inline-flex')[0] as HTMLDivElement;
-            if (difficultyContainer?.parentElement && ratingElement) {
-                difficultyContainer.parentElement.insertBefore(ratingElement, difficultyContainer.parentElement.firstChild);
+        ratingElement.textContent = problem.rating;
+        ratingElement.style.fontSize = '11px';
+        ratingElement.style.letterSpacing = '.5px';
+        ratingElement.style.borderRadius = '6px';
+        ratingElement.style.width = '60px';
+        ratingElement.style.textAlign = 'center';
+        ratingElement.style.padding = '4px 8px';
+        ratingElement.style.transition = 'all 0.2s ease';
+
+        chrome.storage.local.get(['isDarkTheme'], (result) => {
+            const isDark = result.isDarkTheme;
+            if (ratingElement) {
+                ratingElement.style.backgroundColor = isDark ? '#373737' : '#f3f4f5';
+                ratingElement.style.color = isDark ? '#40a9ff' : '#1a1a1a';
+                ratingElement.style.border = `1px solid ${isDark ? 'rgba(255, 255, 255, 0.1)' : 'rgba(0, 0, 0, 0.1)'}`;
             }
         });
+
+        const difficultyContainer = document.querySelectorAll('div.relative.inline-flex')[0] as HTMLDivElement;
+        if (difficultyContainer?.parentElement && ratingElement) {
+            difficultyContainer.parentElement.insertBefore(ratingElement, difficultyContainer.parentElement.firstChild);
+        }
     });
 }
 
diff --git a/src/popup/settings.html b/src/popup/settings.html
@@ -47,9 +47,6 @@
             <button id="show-difficulty-btn" class="material-button settings-btn">
                 <span id="show-difficulty-icon"></span> Show Difficulty
             </button>
-            <button id="show-rating-btn" class="material-button settings-btn">
-                <span id="show-rating-icon"></span> Show Rating
-            </button>
             <div id="review">
                 Leave us a
                 <a target="_blank"
diff --git a/src/popup/settings.ts b/src/popup/settings.ts
@@ -109,11 +109,6 @@ document.addEventListener('DOMContentLoaded', () => {
         const showDifficultyIcon = document.getElementById('show-difficulty-icon');
         if (showDifficultyIcon) showDifficultyIcon.textContent = result.showDifficulty ? '✅' : '❌';
     });
-    
-    chrome.storage.local.get(['showRating'], (result) => {
-        const showRatingIcon = document.getElementById('show-rating-icon');
-        if (showRatingIcon) showRatingIcon.textContent = result.showRating ? '✅' : '❌';
-    });
 
     // Helper function to send messages safely to content script
     function safelySendMessage(message: any) {
@@ -190,16 +185,4 @@ document.addEventListener('DOMContentLoaded', () => {
             });
         });
     });
-
-    const showRatingBtn = document.getElementById('show-rating-btn');
-    showRatingBtn && showRatingBtn.addEventListener('click', function () {
-        chrome.storage.local.get(['showRating'], (result) => {
-            const showRating = !result.showRating;
-            chrome.storage.local.set({ showRating: showRating }, () => {
-                const showRatingIcon = document.getElementById('show-rating-icon');
-                if (showRatingIcon) showRatingIcon.textContent = showRating ? '✅' : '❌';
-                safelySendMessage({ action: 'updateDescription', title: document.title || 'title' });
-            });
-        });
-    });
 });
