Skip to content
This repository has been archived by the owner on Jan 10, 2019. It is now read-only.

Commit

Permalink
update: Updated thanks page showing
Browse files Browse the repository at this point in the history
* The thanks page URL did not have a ?to parameter attached to it which
  caused a bad UX to happen
  • Loading branch information
mrshu committed Aug 22, 2014
1 parent 4b98586 commit c2127d6
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions js/background.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,14 @@ function Background()
var prev_version = localStorage['prev_version'];

if (prev_version === undefined) {
chrome.tabs.create({'url': "https://duckduckgo.com/extensions/thanks/"});
chrome.tabs.create({
'url': "https://duckduckgo.com/extensions/thanks/?to=" + curr_version
});
} else if (prev_version !== curr_version) {
chrome.tabs.create({'url':
"https://duckduckgo.com/extensions/thanks/?from=" + prev_version + "&to="
+ curr_version});
chrome.tabs.create({
'url': "https://duckduckgo.com/extensions/thanks/?from=" + prev_version + "&to="
+ curr_version
});
}
localStorage['prev_version'] = curr_version;

Expand Down

0 comments on commit c2127d6

Please sign in to comment.