Fix expanding/unexapnding read receipts

Make the AccessibleButton element be the img directly rather than
wrapping it in a div which is unnecessary and breaks the rendering.

Fixes https://github.com/vector-im/riot-web/issues/3052

src/components/views/avatars/BaseAvatar.js

@@ -159,13 +159,14 @@ module.exports = React.createClass({
         }
         if (onClick != null) {
             return (
-                <AccessibleButton className="mx_BaseAvatar" onClick={onClick}>
+                <AccessibleButton className="mx_BaseAvatar mx_BaseAvatar_image"
-                    <img className="mx_BaseAvatar_image" src={imageUrl}
+                    element='img'
+                    src={imageUrl}
+                    onClick={onClick}
                     onError={this.onError}
                     width={width} height={height}
                     title={title} alt=""
                     {...otherProps} />
-                </AccessibleButton>
             );
         } else {
             return (

src/components/views/rooms/ReadReceiptMarker.js

@@ -192,9 +192,9 @@ module.exports = React.createClass({
                     width={14} height={14} resizeMethod="crop"
                     style={style}
                     title={title}
+                    onClick={this.props.onClick}
                 />
             </Velociraptor>
         );
-        /* onClick={this.props.onClick} */
     },
 });