debezium.github.io icon indicating copy to clipboard operation
debezium.github.io copied to clipboard

DBZ-2619 Adding few social share buttons

Open uidoyen opened this issue 4 years ago • 12 comments

uidoyen avatar Jan 07 '21 13:01 uidoyen

So I like the idea of share buttons, but can we make them more privacy-friendly? I.e. instead of directly calling those remote URLs, put this behind one more click, so the user consciously opts into this. E.g. see here for one possible solution: https://www.heise.de/extras/socialshareprivacy/ (it's in German, but you'll find similar things described in English, too).

gunnarmorling avatar Jan 08 '21 09:01 gunnarmorling

@gunnarmorling Updated the PR.

uidoyen avatar Feb 04 '21 15:02 uidoyen

Hey @uidoyen, nice work overall! A few remarks:

  • Let's remove Tumblr and Stumble, I don't think they're used that widely any more
  • The letter icon seems oddly stretched, can you double check on this?
  • I reckon FaceBook is a plain link and thus can be shown "enabled" right away?

gunnarmorling avatar Feb 08 '21 11:02 gunnarmorling

Can we do without that permanent feature? We'd need cookie policy disclaimer and option to opt-out for European countries, and it's all not worth the hassle.

gunnarmorling avatar Feb 11 '21 14:02 gunnarmorling

Can we do without that permanent feature? We'd need cookie policy disclaimer and option to opt-out for European countries, and it's all not worth the hassle.

Removed permanent feature.

uidoyen avatar Feb 15 '21 05:02 uidoyen

Hey @uidoyen, nice work overall! A few remarks:

  • Let's remove Tumblr and Stumble, I don't think they're used that widely any more
  • The letter icon seems oddly stretched, can you double check on this?
  • I reckon FaceBook is a plain link and thus can be shown "enabled" right away?

As per https://github.com/panzi/SocialSharePrivacy/issues/50 Facebook is only a link and no request to Facebook is made to display it. Fixed the rest.

uidoyen avatar Feb 15 '21 05:02 uidoyen

Hey @uidoyen, could you rebase this and push it to the preview environment if you got some cycles? @hguerrero was just asking for this feature :)

gunnarmorling avatar Aug 12 '21 20:08 gunnarmorling

@gunnarmorling Done!

uidoyen avatar Aug 19 '21 16:08 uidoyen

Thx! Did you push it to staging, I can't seem to find it there.

gunnarmorling avatar Aug 19 '21 17:08 gunnarmorling

Thx! Did you push it to staging, I can't seem to find it there.

@gunnarmorling Just pushed it to staging

uidoyen avatar Aug 20 '21 11:08 uidoyen

@gunnarmorling @hguerrero Could you please review the staging?

uidoyen avatar Sep 09 '21 14:09 uidoyen

Hi @uidoyen I'm sorry this has fallen through the cracks. If this is still relevant, could you rebase this PR for me? I'll then take a look and review the changes, thanks!

Naros avatar Jul 27 '22 14:07 Naros