-
-
Notifications
You must be signed in to change notification settings - Fork 830
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Redesign: move right panel below room/group header #2260
Conversation
to put a resize handle between the body and the right panel later on
this breaks user view for now but this is not available through any UI as it is, and we don't know yet what this will look like in the new design, even if it will be a feature at all.
Push hook is failing because disabling e2e test didn't work there, see #2261 |
Green! ✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise lgtm
<div className="mx_RoomView_statusAreaBox_line"></div> | ||
{ statusBar } | ||
<MainSplit panel={rightPanel} collapsedRhs={this.props.collapsedRhs}> | ||
<div className={fadableSectionClasses}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have we lost the <main>
/ <aside>
? Would be good to keep these as they help a11y.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. Added it back, but as the root of the group/room view, so the <aside>
is now inside the <main>
. Before it was a sibling of of <main>
, but having it as a child seems to be the defined semantics.
element-hq/element-web#7556