-
-
Notifications
You must be signed in to change notification settings - Fork 829
Conversation
@Fnux can you provide a link to a compiled version of this Riot for our design team to review? |
Here: https://staticwebhosting.ungleich.ch/~fnux/riot-compact/webapp/ |
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.
Principally the proposal seems fine, but I'm getting really bad issues where:
- When toggling on, my timeline spends a few seconds first before it changes layout reliably
- When toggling off, avatars don't re-appear in the timeline
Can we make those 2 behaviours more reliable first before merging?
Ack, I'll patch these two issues at some point in the next few days. |
@Fnux, are you still interested in working on this? |
Similar to my other merge request: I'll eventually complete but a lot of things are happening on my side/work these days so it has been delayed.
I'll have to find a way to reload the displayed room when toggling the setting, but have no clue at the moment. |
Unfortunately this PR is stale at this point as it's gone quite out of sync with trunk. I'm going to have to close it as part of our ongoing efforts to clean up the repository. |
Fixes element-hq/element-web#12998.
useCompactLayout
setting to preferences tab.I'm still not completly convinced by the updated layout: I'll try to gather some feedback here and there and update if needed.
Before:
After: