Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate ShareMeNot functionality #196

Merged
merged 14 commits into from Jun 10, 2014
Merged

Conversation

froeschele
Copy link
Contributor

This branch supports ShareMeNot's social widget replacement functionality. Buttons are replaced if the corresponding domains are blocked by Privacy Badger, and those domains are temporarily unblocked if the button is clicked.

This functionality is enabled by default (Utils.isSocialWidgetReplacementEnabled() just returns true), but should be linked to a user-facing setting.

Test page: http://sharemenot.cs.washington.edu/Test%20Installation.shtml.

Caveat: Button replacement messes with webpage layout on some pages, so this should likely be an opt-in feature.

@cooperq
Copy link
Contributor

cooperq commented Jun 10, 2014

This is awesome, thanks so much! Can you show me an example of a page where it messes with the layout? I will have to do some testing and if it does mess up the layout as you say I will want to make an options page for privacy badger before I merge this pull request.

@froeschele
Copy link
Contributor Author

Hmm, I can't find a great example at the moment. Here's a page where the replacement buttons look weird (scroll to the bottom), but I just realized they look a little weird normally as well: http://www.webmonkey.com/2010/04/adding-facebook-like-buttons-to-your-site-is-damn-easy/. I do occasionally get complaints from ShareMeNot users.

@cooperq
Copy link
Contributor

cooperq commented Jun 10, 2014

It looks like that site is wonky with or without ShareMeNot, I will keep
testing and see if I can find any other weird behavior. One thing I
noticed is that the webmonkey page seems to have links to share on digg
and stumble upon that don't get replaced. Is that the expected behavior
or a bug?

  • Cooper

On 06/09/2014 10:10 PM, Franzi Roesner wrote:

Hmm, I can't find a great example at the moment. Here's a page where the
replacement buttons look weird (scroll to the bottom), but I just
realized they look a little weird normally as well:
http://www.webmonkey.com/2010/04/adding-facebook-like-buttons-to-your-site-is-damn-easy/.
I do occasionally get complaints from ShareMeNot users.


Reply to this email directly or view it on GitHub
#196 (comment).

@froeschele
Copy link
Contributor Author

I think those buttons don't involve third-party requests, just local css,
so there's actually nothing to block. However, it can be the case that
ShareMeNot will sometimes miss unfamiliar versions of buttons (i.e.,
buttons not represented in socialwidgets.json). If the requests for those
buttons are blocked by Privacy Badger, the buttons should just be missing,
though.

On Tue, Jun 10, 2014 at 11:53 AM, Cooper Quintin notifications@github.com
wrote:

It looks like that site is wonky with or without ShareMeNot, I will keep
testing and see if I can find any other weird behavior. One thing I
noticed is that the webmonkey page seems to have links to share on digg
and stumble upon that don't get replaced. Is that the expected behavior
or a bug?

  • Cooper

On 06/09/2014 10:10 PM, Franzi Roesner wrote:

Hmm, I can't find a great example at the moment. Here's a page where the
replacement buttons look weird (scroll to the bottom), but I just
realized they look a little weird normally as well:

http://www.webmonkey.com/2010/04/adding-facebook-like-buttons-to-your-site-is-damn-easy/
.
I do occasionally get complaints from ShareMeNot users.


Reply to this email directly or view it on GitHub
<
#196 (comment)
.


Reply to this email directly or view it on GitHub
#196 (comment)
.

@cooperq
Copy link
Contributor

cooperq commented Jun 10, 2014

Cool, that makes sense. This looks fine so far in my testing. I will add an option to turn it off when I make an options page for privacy badger.

cooperq added a commit that referenced this pull request Jun 10, 2014
Integrate ShareMeNot functionality. Fixes #84
@cooperq cooperq merged commit 4abeec7 into EFForg:master Jun 10, 2014
@ghostwords ghostwords added the widgets Click-to-activate placeholders for blocked but potentially useful social buttons/widgets label Jul 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
widgets Click-to-activate placeholders for blocked but potentially useful social buttons/widgets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants