Skip to content

Commit a190209

Browse files
Merge pull request #190 from jdi-testing/issue_189
add onUpdate condition
2 parents ce75ad2 + 57d84fe commit a190209

3 files changed

Lines changed: 12 additions & 10 deletions

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "jdi-react-extension",
3-
"version": "3.0.11",
3+
"version": "3.0.12",
44
"description": "jdi react extension",
55
"scripts": {
66
"start": "npm run webpack",

src/js/blocks/autoFind/autoFindProvider/pageDataHandlers.js

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -70,14 +70,16 @@ export const highlightElements = (
7070
};
7171

7272
export const setUrlListener = (onHighlightOff) => {
73-
chrome.tabs.onUpdated.addListener((tabId, changeinfo, tab) => {
74-
if (changeinfo && changeinfo.status === "complete") {
75-
urlListenerScriptExists = false;
76-
generationScriptExists = false;
77-
port = null;
78-
onHighlightOff();
79-
}
80-
});
73+
getPageId((currentTabId) =>
74+
chrome.tabs.onUpdated.addListener((tabId, changeinfo, tab) => {
75+
if (changeinfo && changeinfo.status === "complete" && currentTabId === tabId) {
76+
urlListenerScriptExists = false;
77+
generationScriptExists = false;
78+
port = null;
79+
onHighlightOff();
80+
}
81+
})
82+
);
8183

8284
if (!urlListenerScriptExists) {
8385
runContentScript(urlListener, () => {

src/manifest.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
"name": "JDN",
44
"description": "",
55
"devtools_page": "index.html",
6-
"version": "3.0.11",
6+
"version": "3.0.12",
77
"permissions": [
88
"activeTab",
99
"tabs",

0 commit comments

Comments
 (0)