If you share your statistics via facebook, you get a german headline and an english description

[passbrains]

Adapted from ​https://www.passbrains.com/dashboard/view-ticket.php?ticket_no=ASA-11

Environment

Chrome/Opera/Safari

How to reproduce

1. Install ABP
2. Click at the icon in Safari
3. Click at Share via Facebook

Observed behaviour

But the headline is in German, text in English

Expected behaviour

That the text is in English OR in German

[passbrains]

1 - 12 Aug 2014 13:52:12 posted by Philip Hill
It's not Safari-specific, happens also with Google Chrome.

[saroyanm]

The description part of facebook share dialog is being taken from "​https://adblockplus.org/".
So I can see 3 solutions here:

1. Use description from web locale meta (14 Languages), detect language and use url for appropriate language (ex.: ​https://adblockplus.org/de/).
2. Add share dialog description to ABP locales and load that description from ABP (48 locales).
3. Use only English text (which I don't think we want here).

Not sure which is more preferable.

With 1-st solution I've noticed problem that even when using localized sharing link the description is always in English, not yet sure the case, this something connected to website itself, so for example if "​https://adblockplus.org/de/chrome" url is being added to the post in FB, it's always shows English text not sure if it's a bug or feature. Also in case of 1-st solution we don't have all ABP translations for website.

[sebastian]

I think it makes sense to have Facebook use the title and description from the metadata on our website, and to fix those if necessary.

Consider that the FB button inside the extension's bubble UI, isn't the only way to share ablockplus.org on Facebook. So we should make sure that those metadata are correct and complete.

Or do I miss something here?

[saroyanm]

Replying to sebastian:

I think it makes sense to have Facebook use the title and description from the metadata on our website, and to fix those if necessary.

Yes anyway it make sense to fix the metadata because if you will try to share for example current link ​https://adblockplus.org/de/chrome post description is in English again, while we have correct metadata in German version of the website, so not sure why it takes again English text from ​https://adblockplus.org/en/chrome.
I think we need to create another ticket for website fix.
And maybe after fix we will need to generate correct link according to the platform language and use English link for the missing website languages.

[saroyanm]

Replying to sebastian:
I've created another ticket for fixing metadata issue in website:
https://issues.adblockplus.org/ticket/1274

[sebastian]

Thanks. I was just about to do it myself. :)

[juliandoucette]

This bug should be fixed by #5447.

[juliandoucette]

Scratch that. It looks like the locale is not being sent to adblockplus.org. Can you look into this greiner?

[greiner]

Replying to juliandoucette:

Scratch that. It looks like the locale is not being sent to adblockplus.org. Can you look into this greiner?

I guess you're referring to the fb_locale query string parameter?! Because ​their documentation mentions

When Facebook needs to render an object in one of the specified locales, we will make a request to the object URL with the fb_locale URL parameter set, e.g. &fb_locale=es_ES. The app should then return the object markup in that specified locale.

If so, it sounds like we'd have to handle this parameter in our server-side routing logic because ​https://adblockplus.org/?fb_locale=de_DE currently ​returns the English version.

[agiammarchi]

We are removing share stats https://issues.adblockplus.org/ticket/5613 so I wonder if this ticket makes sense now.

[greiner]

It's indeed no longer a concern for UI since we're going to remove share functionality from both the icon popup as well as the first-run page.

The underlying issue still remains though. Facebook's crawler will fetch the English version of the page whenever someone shares a link to adblockplus.org. Therefore I'll set the module to Websites.

[juliandoucette]

I believe that the websites part of this issue was resolved by #5683?

[greiner]

Indeed sounds like there's nothing to do anymore. I'm closing it as "invalid" then.