Rob--W / stackexchange-notifications

Realtime desktop notifications for Stack Exchange sites.
44 stars 8 forks source link

Fix initial link setting #10

Closed kav2k closed 10 years ago

kav2k commented 10 years ago

Without this fix, on extension initialization:

localStorage['open-on-click'] === undefined localStorage.getItem('open-on-click') === null

JavaScript, in its infinite wisdom, assigns a string "null" here: link.defaultValue = link.value = bg.getLink();

Therefore, upon opening the options page the link field is filled with "null", and if options are saved the default link is prevented from being used.

Rob--W commented 10 years ago

Thanks!