From b561b711df08b58dec89b02a1b8aacdec10c89c4 Mon Sep 17 00:00:00 2001 From: David Baker Date: Thu, 2 Feb 2017 17:36:26 +0000 Subject: [PATCH] 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 | 15 ++++++++------- src/components/views/rooms/ReadReceiptMarker.js | 2 +- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/src/components/views/avatars/BaseAvatar.js b/src/components/views/avatars/BaseAvatar.js index c9c84aa1bf..5862a9d4f1 100644 --- a/src/components/views/avatars/BaseAvatar.js +++ b/src/components/views/avatars/BaseAvatar.js @@ -159,13 +159,14 @@ module.exports = React.createClass({ } if (onClick != null) { return ( - - - + ); } else { return ( diff --git a/src/components/views/rooms/ReadReceiptMarker.js b/src/components/views/rooms/ReadReceiptMarker.js index 9744ea8914..230efbd1ea 100644 --- a/src/components/views/rooms/ReadReceiptMarker.js +++ b/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} /> ); - /* onClick={this.props.onClick} */ }, });