Avatar alt isn't attribute-escaped, nor translated

[nbachiyski]

In [8661] the avatar alt default value is Avatar and cannot be translated.

Also, if the avatar alt contains ' or tags, it will break the HTML code.

[ryan]

(In [9076]) Avatar alt fixes from nbachiyski. fixes #7536

[matt]

Giving every avatar a default alt text of "avatar" is very bad accesibility, I can't imagine anything more annoying to a screen reader.

​http://en.wikipedia.org/wiki/Alt_attribute

"While the use of meaningful alt text is necessary to comply with accessibility standards, and is good practice, sometimes an image is used for purely decorative purposes. In this case, one should use an empty alt attribute (alt="")."

[ShaneF]

Replying to matt:

Giving every avatar a default alt text of "avatar" is very bad accesibility, I can't imagine anything more annoying to a screen reader.

Agree.

[westi]

Although I introduced this for #7523 I agree with matt here that the default should be empty.

Having a page full of Avatars being read out would get a little frustrating when viewing comments.

[westi]

(In [9127]) Default to an empty alt attribute on Avatars. Fixes #7536.

[Speedboxer]

Change phpdoc to say that the default is blank

[Speedboxer]

The phpdoc text needs to be changed to say the default is blank.

[ryan]

(In [9200]) get_avatar() phpdoc update from Speedboxer. fixes #7536