Skip to content

Commit 7d222cf

Browse files
author
Mihir
committed
Fix hover mode
- Refresh the page every time hover mode or percentage mode is toggled
1 parent a39dfb3 commit 7d222cf

File tree

3 files changed

+20
-8
lines changed

3 files changed

+20
-8
lines changed

src/main.css

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,16 @@
5454
box-shadow: 0 1px 1px rgba(0, 0, 0, 0.06) !important;
5555
}
5656

57+
/* Hover mode - hide badge by default */
58+
.affordable-badge.hover-mode {
59+
display: none !important;
60+
}
61+
62+
/* Show badge on parent hover when in hover mode */
63+
.affordable-hover-container:hover .affordable-badge.hover-mode {
64+
display: inline-flex !important;
65+
}
66+
5767
.affordable-badge__swatch {
5868
width: 6px !important;
5969
height: 0.9em !important;

src/main.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -355,15 +355,10 @@ function append(elementInfo, element, productPrice, settings, isPriceRange) {
355355
desiredElement.appendChild(badge);
356356

357357
if (settings.hoverMode) {
358-
// Hover Mode Attributes
359-
badge.style.display = 'none';
360-
desiredElement.addEventListener('mouseover', function handleMouseOver() {
361-
badge.style.display = 'inline-flex';
362-
});
358+
// Hover Mode - use CSS classes for visibility control
359+
badge.classList.add('hover-mode');
360+
desiredElement.classList.add('affordable-hover-container');
363361
desiredElement.setAttribute("title", `It will take you ${getTimeTakenToEarn(productPrice, settings.salary)} to earn ${productPrice}`);
364-
desiredElement.addEventListener('mouseout', function handleMouseOut() {
365-
badge.style.display = 'none';
366-
});
367362
} else if (!isPriceRange) {
368363
badge.setAttribute("style", "display:inline-flex");
369364
desiredElement.setAttribute("title", `It will take you ${getTimeTakenToEarn(productPrice, settings.salary)} to earn ${productPrice}`);
@@ -444,4 +439,9 @@ function undoUpdates() {
444439
element.classList.remove("affordable-highlight-tertiary")
445440
})
446441

442+
elements = document.querySelectorAll(".affordable-hover-container")
443+
elements.forEach((element) => {
444+
element.classList.remove("affordable-hover-container")
445+
})
446+
447447
}

src/settingsPage.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,7 @@ function addHoverModeUpdateListener() {
135135
chrome.storage.sync.set({ settings });
136136
})
137137
sendMessage()
138+
chrome.tabs.reload();
138139
});
139140
}
140141

@@ -197,6 +198,7 @@ function addPercentageModeListener() {
197198
chrome.storage.sync.set({ settings });
198199
})
199200
sendMessage()
201+
chrome.tabs.reload();
200202
});
201203
}
202204

0 commit comments

Comments
 (0)