kevinbazira / algori-social-share-buttons-lite Goto Github PK

Hello,

I am using your plugin for some of my Wordpress projects. On a normal site (non-multisite) it works flawlessly. With a Multisite setup there seems to be a problem:

When the social share buttons are used on one of the network sites as a user without super admin privileges (so a normal editor for example) after saving the post which contains your block I get a "Block validation" error in the console after reloading the page. It seems like the links to the different Social platforms won't get saved properly.

This is the error message I get:

Block validation: Block validation failed for cgb/block-algori-social-share-buttons

Expected:

<div style="text-align:center" class="wp-block-cgb-block-algori-social-share-buttons"> <button class="bttn-simple bttn-md bttn-primary algori-social-share-buttons-settings algori-social-share-buttons-facebook" onclick="window.open('https://www.facebook.com/sharer.php?u=http://localhost/project/article/', '_blank')"><i class="fab fa-facebook-f"></i>   Facebook</button></div>

Actual:

<div style="text-align:center" class="wp-block-cgb-block-algori-social-share-buttons"> <button class="bttn-simple bttn-md bttn-primary algori-social-share-buttons-settings algori-social-share-buttons-facebook"><i class="fab fa-facebook-f"></i>   Facebook</button></div>

As you can see the button onclick handler won't get saved properly WHEN I do not have super admin privileges in a Multisite setup. As a super admin I can save and reload everything just fine.

First of all, thanks for creating this cool Plugin!

We found one problem while using the email sharing button. This creates a link which follows this pattern:

mailto:body=<LINK_TO_ARTICLE>

This link opens the mail client and prefills the recipient field with everything after mailto:. The desired behavior would be to only have the link to the article in the mail body whilst the recipient field remains empty. This can be achieved if the mailto link has a ? between the : and the link to the article.
Therefore, the pattern should be:

mailto:?body=<LINK_TO_ARTICLE>

UX wise it would be better if native apps (whatsapp, facebook, etc.) were used for sharing (if installed).

For whatsapp this pattern can be used for mobile devices:
WhatApp: https://wa.me/?text=urlencodedtext

Maybe there are similar options for other apps too.