mozilla / lightbeam-we

Web Extension version of the Firefox Lightbeam add-on
https://addons.mozilla.org/en-GB/firefox/addon/lightbeam/
Mozilla Public License 2.0
179 stars 61 forks source link

Properly munge inbound data in all cases. Fixes #187. #188

Closed biancadanforth closed 7 years ago

biancadanforth commented 7 years ago

Dynamic vars were not updating correctly with the first party count in the switch statement for the isVisible and firstParty cases in store.setWebsite.

biancadanforth commented 7 years ago

@jonathanKingston , there might be a more elegant way to do this, but this works. :)

biancadanforth commented 7 years ago

@jonathanKingston , made changes as we discussed in slack. It's much cleaner now. Let me know what you think!