From aefecfc645aff7eab5c57415bdd1f17d3534c25c Mon Sep 17 00:00:00 2001
From: Matthew Hodgson <matthew@matrix.org>
Date: Fri, 15 Jan 2016 15:23:41 +0000
Subject: [PATCH] tweak roomheader layout when editing

---
 src/components/views/rooms/RoomHeader.js      | 8 +++++---
 src/components/views/settings/ChangeAvatar.js | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/components/views/rooms/RoomHeader.js b/src/components/views/rooms/RoomHeader.js
index 41b1b364e2..79eac7b252 100644
--- a/src/components/views/rooms/RoomHeader.js
+++ b/src/components/views/rooms/RoomHeader.js
@@ -213,7 +213,7 @@ module.exports = React.createClass({
             if (this.props.room) {
                 if (this.props.editing) {
                     roomAvatar = (
-                        <div onClick={ this.onAvatarPickerClick }>
+                        <div className="mx_RoomHeader_avatarPicker" onClick={ this.onAvatarPickerClick }>
                             <ChangeAvatar room={this.props.room} showUploadSection={false} width={48} height={48} />
                             <div className="mx_RoomHeader_avatarPicker_edit">
                                 <label htmlFor="avatarInput" ref="file_label">
@@ -228,7 +228,9 @@ module.exports = React.createClass({
                 }
                 else {
                     roomAvatar = (
-                        <RoomAvatar room={this.props.room} width="48" height="48" />
+                        <div onClick={this.props.onSettingsClick}>
+                            <RoomAvatar room={this.props.room} width="48" height="48"/>
+                        </div>
                     );
                 }
             }
@@ -279,7 +281,7 @@ module.exports = React.createClass({
         }
 
         return (
-            <div className="mx_RoomHeader">
+            <div className={ "mx_RoomHeader " + (this.props.editing ? "mx_RoomHeader_editing" : "") }>
                 { header }
             </div>
         );
diff --git a/src/components/views/settings/ChangeAvatar.js b/src/components/views/settings/ChangeAvatar.js
index 7f19b193f7..7bd3d7754b 100644
--- a/src/components/views/settings/ChangeAvatar.js
+++ b/src/components/views/settings/ChangeAvatar.js
@@ -39,7 +39,7 @@ module.exports = React.createClass({
     getDefaultProps: function() {
         return {
             showUploadSection: true,
-            className: "mx_Dialog_content", // FIXME - shouldn't be this by default
+            className: "",
             width: 80,
             height: 80,
         };