Issue 6082 - Update suport section in the help tab

Issue 6082 - Update suport section in the help tab

[saroyanm, 2017-11-23 11:46:05]

@Martin, @Thomas: This are again small changes, whoever will have time to have
look would be great.

[Thomas Greiner, 2017-11-23 15:02:17]

https://codereview.adblockplus.org/29616555/diff/29616556/desktop-options.html
File desktop-options.html (right):

https://codereview.adblockplus.org/29616555/diff/29616556/desktop-options.htm...
desktop-options.html:324: </li>
It's a pity that we're left with only the email link now. Didn't we want to
refer to our issue reporter or link to https://adblockplus.org/en/bugs?

https://codereview.adblockplus.org/29616555/diff/29616556/desktop-options.htm...
desktop-options.html:342: <p>
This paragraph no longer exists according to the spec.

[saroyanm, 2017-11-23 16:04:32]

https://codereview.adblockplus.org/29616555/diff/29616556/desktop-options.html
File desktop-options.html (right):

https://codereview.adblockplus.org/29616555/diff/29616556/desktop-options.htm...
desktop-options.html:324: </li>
On 2017/11/23 15:02:17, Thomas Greiner wrote:
> It's a pity that we're left with only the email link now. Didn't we want to
> refer to our issue reporter or link to https://adblockplus.org/en/bugs

Seems like the specification is updated ->
https://bitbucket.org/adblockplus/spec/commits/94241ac3c6abdd4a9b749682e58bc9...

https://codereview.adblockplus.org/29616555/diff/29616556/desktop-options.htm...
desktop-options.html:342: <p>
On 2017/11/23 15:02:17, Thomas Greiner wrote:
> This paragraph no longer exists according to the spec.

Interesting, I'll remove it.

[saroyanm, 2017-11-23 17:49:04]

New patch uploaded.

https://codereview.adblockplus.org/29616555/diff/29616594/locale/en_US/deskto...
File locale/en_US/desktop-options.json (right):

https://codereview.adblockplus.org/29616555/diff/29616594/locale/en_US/deskto...
locale/en_US/desktop-options.json:360: "message": "Think content or
functionality is being incorrectly blocked by us? Email:
<a>support@adblockplus.org</a>"
On one hand, having "text ... Email: <a></a>" will provide translators more
flexibility in order of the translation on another hand we might need to
translate word current string which include string "Email:" again.

[saroyanm, 2017-11-23 17:56:19]

https://codereview.adblockplus.org/29616555/diff/29616594/locale/en_US/deskto...
File locale/en_US/desktop-options.json (right):

https://codereview.adblockplus.org/29616555/diff/29616594/locale/en_US/deskto...
locale/en_US/desktop-options.json:360: "message": "Think content or
functionality is being incorrectly blocked by us? Email:
<a>support@adblockplus.org</a>"
On 2017/11/23 17:49:04, saroyanm wrote:
> On one hand, having "text ... Email: <a></a>" will provide translators more
> flexibility in order of the translation on another hand we might need to
> translate word current string which include string "Email:" again.

I have uploaded another patch, which keep the translation for "Email:". Not sure
which option is best in the end:/ Still.. thinking.

[saroyanm, 2017-11-24 13:37:49]

https://codereview.adblockplus.org/29616555/diff/29616598/locale/en_US/deskto...
File locale/en_US/desktop-options.json (right):

https://codereview.adblockplus.org/29616555/diff/29616598/locale/en_US/deskto...
locale/en_US/desktop-options.json:360: "message": "Think content or
functionality is being incorrectly blocked by us?"
As discussed internally: As we would like to remove the Easylist reference ASAP,
but we are not finilized current text, we will remove this text completely for
now and only leave email to the support instead, until the text will be
finalized and updated.

[saroyanm, 2017-11-24 13:50:35]

Ready for the review

[Thomas Greiner, 2017-11-24 14:14:58]

LGTM but note the typo in the review title