privly / privly-firefox

Official Privly Browser Extension for Firefox - Allows for Viewing and Posting Content on Any Website Without the Host Site Having Access
https://priv.ly
MIT License
21 stars 15 forks source link

Make Privly a restartless extension #8

Closed ehsan closed 9 years ago

ehsan commented 12 years ago

Starting from Firefox 4, Firefox supports extensions which can be installed/removed without restarting.

This mainly boils down to ensuring that the dynamic changes to the browser window happen on extension startup, and the extension is responsible for reverting those changes at shutdown.

I have started to make Privly restartless in this branch: https://github.com/ehsan/privly-firefox/tree/restartless

The major todo item is hooking up the Options dialog, but the rest of the basics should work fine. I didn't have a lot of time to hack on this today, so consider this branch as the skeleton. In particular I have not performed any testing as I don't have a bit.ly account yet (I've submitted a request for an invitation!).

Please feel free to pull in this branch, and build on top of it, or give me access to priv.ly so that I can test this and prepare it for a pull request myself.

smcgregor commented 12 years ago

@ehsan Balaji is cleaning up observers.js and will comment on it later tonight. Refactoring it into a module would be great.

It looks like your gmail may have gotten shut down (!), do you have an email address you want an invite sent to?

ehsan commented 12 years ago

Hmm, I thought that my gmail problem has been fixed! Could you please use my first name at mozilla.com? Also, I'd appreciate if you can forward the gmail bounce message to that address as well! :-)

smcgregor commented 12 years ago

I sent the invitation to your Mozilla email. Please let me know if you have any problems.

They reactivated your gmail already? I read about it being shut down on your blog, so I didn't send an invite there. I can issue another invite to the gmail if you want me to.

ehsan commented 12 years ago

Ah, yes, the account is reactivated. But thanks, the invitation you already sent should be enough.

smcgregor commented 11 years ago

Since we are prepping to add a compiled library to the extension, now is the time to decide whether we are going to make the extension restartless. I am not convinced that making the extension restartless is worth the tradeoffs. However, the potential benefits of Jetpack could tip the balance. My notes are below, but the conclusion is open to discussion. I would certainly revisit the conclusion if someone at Mozilla made a recommendation ;)

Just making the extension Restartless

Pros:

Cons:

Moving to Jetpack

Pros:

Cons:

Conclusion

For me, this comes down to the direction Mozilla is moving with their extension support. If Jetpack extensions will be increasingly dominant, we should move to Jetpack at our earliest convenience. Otherwise, we should probably stick with a restart-dependent add-on.

smcgregor commented 9 years ago

Subsumed by #38.