danwent / Perspectives

Perspectives Firefox Extension
http://perspectives-project.org
66 stars 19 forks source link

Perspectives icon continually spinning #113

Closed daveschaefer closed 10 years ago

daveschaefer commented 10 years ago

Bug report via email:

Sometimes (not always) I see the extension's icon in the toolbar changing to a circling "loading" icon even if I deactivate automatic checking of sites, and even if visiting non-https sites.

Repro steps:

daveschaefer commented 10 years ago

I haven't been able to reproduce this myself but I may have discovered a fix nonetheless, while looking at the code and fixing #112 . Currently Perspectives sets the icon spinning when it receives any OnLocationChange event from firefox, even for non-https sites. The icon is then set to another status by other events in the browser. We should change the code to only start spinning for https sites (when we know we'll be sending a query). That may fix this bug.

Checking the scheme for https inside all OnLocationChange events may be a good idea anyway - we may be able to send off the queries at that time, if they haven't been sent off already, thus starting queries sooner, getting results sooner, and moving one step towards getting notary results before the actual connection is made.

daveschaefer commented 10 years ago

A patch for this has now been released as part of v4.5! Closing this ticket.

daveschaefer commented 10 years ago

The reporter confirmed that this fix works. Huzzah!