Fixing issue with long issues and some ui fixes

capfei · capfei · commit 0b417de9529f · 2025-12-10T11:43:29.000-08:00
diff --git a/background.js b/background.js
@@ -53,9 +53,14 @@ let currentScanFilter = FILTER_OPTIONS.BOTH;
 function normalizeFilter(value) {
   return Object.values(FILTER_OPTIONS).includes(value) ? value : FILTER_OPTIONS.BOTH;
 }
-try { chrome.action.setPopup({ popup: 'popup.html' }); } catch (err) { console.warn('Popup attach failed:', err); }
+try {
+  chrome.action.setPopup({ popup: 'popup.html' });
+} catch (err) {
+  console.warn('Popup attach failed:', err);
+}
 
-let originTabId; // Store the ID of the tab that initiated the license check
+// Store the ID of the tab that initiated the license check
+let originTabId;
 const dmp = new DiffMatchPatch();
 // In-memory cache of license term-frequency vectors (populated on demand)
 let licenseVectorsCache = null; // { license_key: { word: freq, ... } }
@@ -65,13 +70,17 @@ const activeScans = new Set();
 
 // Database version - increment when structure changes
 const DB_VERSION = 2;
-
+let dbInstance = null;
+let dbOpenPromise = null;
 /**
  * Open (and create/upgrade) the IndexedDB database.
  * @returns {Promise<IDBDatabase>}
  */
 function openDatabase() {
-  return new Promise((resolve, reject) => {
+  if (dbInstance) return Promise.resolve(dbInstance);
+  if (dbOpenPromise) return dbOpenPromise;
+
+  dbOpenPromise = new Promise((resolve, reject) => {
     const request = indexedDB.open('LicenseDB', DB_VERSION);
 
     request.onupgradeneeded = (event) => {
@@ -91,13 +100,23 @@ function openDatabase() {
     };
 
     request.onsuccess = (event) => {
-      resolve(event.target.result);
+      dbInstance = event.target.result;
+      dbInstance.onclose = () => { dbInstance = null; dbOpenPromise = null; };
+      dbInstance.onversionchange = () => {
+        dbInstance?.close();
+        dbInstance = null;
+        dbOpenPromise = null;
+      };
+      resolve(dbInstance);
     };
 
     request.onerror = (event) => {
+      dbOpenPromise = null;
       reject(`IndexedDB error: ${event.target.errorCode}`);
     };
   });
+
+  return dbOpenPromise;
 }
 
 /**
@@ -504,7 +523,7 @@ async function loadAllVectors() {
   return licenseVectorsCache;
 }
 
-// === Added caches for advanced similarity metrics ===
+// Added caches for advanced similarity metrics
 let docFreqCache = null;          // { term: documentFrequency }
 let totalDocsCache = 0;
 const licenseShingleCache = {};   // { license_key: Set<string> }
diff --git a/content.css b/content.css
@@ -37,13 +37,17 @@
   font-size: 22px;
   cursor: pointer;
   transition: all 0.2s ease;
-  border-radius: 50%;
   width: 32px;
   height: 32px;
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  opacity: 0.7;
+  min-width: 32px;
+  min-height: 32px;
+  max-width: 32px;
+  max-height: 32px;
+  border-radius: 50%;
+  box-sizing: border-box;
+  padding: 0;
+  line-height: 1;
+  flex: 0 0 auto;
 }
 
 #license-diff-ui .license-diff-close:hover {
@@ -67,6 +71,7 @@
 
 #license-diff-theme select {
   padding: 4px 22px 4px 8px;
+  width: auto;
   border-radius: 8px;
   border: 1px solid rgba(0, 0, 0, 0.15);
   font-size: 12px;
diff --git a/options.html b/options.html
@@ -183,7 +183,7 @@
   </style>
 </head>
 <body>
-  <h1>LicenseDB Diff Options</h1>
+  <h1>LicenseDB License Diff Options</h1>
 
   <div class="card">
     <h2>Database Information</h2>
