New-style reply design is harder to grok #18194
Assignees
Labels
Comments
|
matrix-org/matrix-react-sdk#6396 is the PR to bring back the avatars (while still keeping replies compact, of course) |
|
Thanks @robintown 🙂 I feel like there is still something to it that feels off. It's blending too well with the current spacing or rhythm. I added some full-size screenshots so I can better flip through and compare. |
|
Hmm, I would suggest we wait for matrix-org/matrix-react-sdk#6396 to get merged and iterate from there onward |
|
Hmm, could giving replies the same lightened color as blockquotes be a solution?
|
|
That looks great to me! Also we could then add a hover effect |
|
Maybe add it to the PR then :D |
Closed
su-ex
added a commit
to SchildiChat/element-desktop
that referenced
this issue
Dec 6, 2022
* Update to Electron 21 ([\element-hq#458](element-hq#458)). Fixes element-hq/element-web#23783. * Further improve replies ([\#6396](matrix-org/matrix-react-sdk#6396)). Fixes element-hq/element-web#19074, element-hq/element-web#18194 element-hq/element-web#18027 and element-hq/element-web#19179. * Enable users to join group calls from multiple devices ([\#9625](matrix-org/matrix-react-sdk#9625)). * fix(visual): make cursor a pointer for summaries ([\#9419](matrix-org/matrix-react-sdk#9419)). Contributed by @r00ster91. * Add placeholder for rich text editor ([\#9613](matrix-org/matrix-react-sdk#9613)). * Consolidate public room search experience ([\#9605](matrix-org/matrix-react-sdk#9605)). Fixes element-hq/element-web#22846. * New password reset flow ([\#9581](matrix-org/matrix-react-sdk#9581)). Fixes element-hq/element-web#23131. * Device manager - add tooltip to device details toggle ([\#9594](matrix-org/matrix-react-sdk#9594)). * sliding sync: add lazy-loading member support ([\#9530](matrix-org/matrix-react-sdk#9530)). * Limit formatting bar offset to top of composer ([\#9365](matrix-org/matrix-react-sdk#9365)). Fixes element-hq/element-web#12359. Contributed by @owi92. * Fix issues around up arrow event edit shortcut ([\#9645](matrix-org/matrix-react-sdk#9645)). Fixes element-hq/element-web#18497 and element-hq/element-web#18964. * Fix search not being cleared when clicking on a result ([\#9635](matrix-org/matrix-react-sdk#9635)). Fixes element-hq/element-web#23845. * Fix screensharing in 1:1 calls ([\#9612](matrix-org/matrix-react-sdk#9612)). Fixes element-hq/element-web#23808. * Fix the background color flashing when joining a call ([\#9640](matrix-org/matrix-react-sdk#9640)). * Fix the size of the 'Private space' icon ([\#9638](matrix-org/matrix-react-sdk#9638)). * Fix reply editing in rich text editor (https ([\#9615](matrix-org/matrix-react-sdk#9615)). * Fix thread list jumping back down while scrolling ([\#9606](matrix-org/matrix-react-sdk#9606)). Fixes element-hq/element-web#23727. * Fix regression with TimelinePanel props updates not taking effect ([\#9608](matrix-org/matrix-react-sdk#9608)). Fixes element-hq/element-web#23794. * Fix form tooltip positioning ([\#9598](matrix-org/matrix-react-sdk#9598)). Fixes element-hq/element-web#22861. * Extract Search handling from RoomView into its own Component ([\#9574](matrix-org/matrix-react-sdk#9574)). Fixes element-hq/element-web#498. * Fix call splitbrains when switching between rooms ([\#9692](matrix-org/matrix-react-sdk#9692)).
su-ex
added a commit
to SchildiChat/element-web
that referenced
this issue
Dec 6, 2022
* Further improve replies ([\element-hq#6396](matrix-org/matrix-react-sdk#6396)). Fixes element-hq#19074, element-hq#18194 element-hq#18027 and element-hq#19179. * Enable users to join group calls from multiple devices ([\element-hq#9625](matrix-org/matrix-react-sdk#9625)). * fix(visual): make cursor a pointer for summaries ([\element-hq#9419](matrix-org/matrix-react-sdk#9419)). Contributed by @r00ster91. * Add placeholder for rich text editor ([\element-hq#9613](matrix-org/matrix-react-sdk#9613)). * Consolidate public room search experience ([\element-hq#9605](matrix-org/matrix-react-sdk#9605)). Fixes element-hq#22846. * New password reset flow ([\element-hq#9581](matrix-org/matrix-react-sdk#9581)). Fixes element-hq#23131. * Device manager - add tooltip to device details toggle ([\#9594](matrix-org/matrix-react-sdk#9594)). * sliding sync: add lazy-loading member support ([\element-hq#9530](matrix-org/matrix-react-sdk#9530)). * Limit formatting bar offset to top of composer ([\element-hq#9365](matrix-org/matrix-react-sdk#9365)). Fixes element-hq#12359. Contributed by @owi92. * Fix issues around up arrow event edit shortcut ([\element-hq#9645](matrix-org/matrix-react-sdk#9645)). Fixes element-hq#18497 and element-hq#18964. * Fix search not being cleared when clicking on a result ([\element-hq#9635](matrix-org/matrix-react-sdk#9635)). Fixes element-hq#23845. * Fix screensharing in 1:1 calls ([\element-hq#9612](matrix-org/matrix-react-sdk#9612)). Fixes element-hq#23808. * Fix the background color flashing when joining a call ([\element-hq#9640](matrix-org/matrix-react-sdk#9640)). * Fix the size of the 'Private space' icon ([\element-hq#9638](matrix-org/matrix-react-sdk#9638)). * Fix reply editing in rich text editor (https ([\element-hq#9615](matrix-org/matrix-react-sdk#9615)). * Fix thread list jumping back down while scrolling ([\element-hq#9606](matrix-org/matrix-react-sdk#9606)). Fixes element-hq#23727. * Fix regression with TimelinePanel props updates not taking effect ([\element-hq#9608](matrix-org/matrix-react-sdk#9608)). Fixes element-hq#23794. * Fix form tooltip positioning ([\element-hq#9598](matrix-org/matrix-react-sdk#9598)). Fixes element-hq#22861. * Extract Search handling from RoomView into its own Component ([\element-hq#9574](matrix-org/matrix-react-sdk#9574)). Fixes element-hq#498. * Fix call splitbrains when switching between rooms ([\element-hq#9692](matrix-org/matrix-react-sdk#9692)). * [Backport staging] Fix replies to emotes not showing as inline ([\element-hq#9708](matrix-org/matrix-react-sdk#9708)).
su-ex
added a commit
to SchildiChat/matrix-react-sdk
that referenced
this issue
Dec 6, 2022
* Further improve replies ([\matrix-org#6396](matrix-org#6396)). Fixes element-hq/element-web#19074, element-hq/element-web#18194 element-hq/element-web#18027 and element-hq/element-web#19179. * Enable users to join group calls from multiple devices ([\matrix-org#9625](matrix-org#9625)). * fix(visual): make cursor a pointer for summaries ([\matrix-org#9419](matrix-org#9419)). Contributed by @r00ster91. * Add placeholder for rich text editor ([\matrix-org#9613](matrix-org#9613)). * Consolidate public room search experience ([\matrix-org#9605](matrix-org#9605)). Fixes element-hq/element-web#22846. * New password reset flow ([\matrix-org#9581](matrix-org#9581)). Fixes element-hq/element-web#23131. * Device manager - add tooltip to device details toggle ([\matrix-org#9594](matrix-org#9594)). * sliding sync: add lazy-loading member support ([\matrix-org#9530](matrix-org#9530)). * Limit formatting bar offset to top of composer ([\matrix-org#9365](matrix-org#9365)). Fixes element-hq/element-web#12359. Contributed by @owi92. * Fix issues around up arrow event edit shortcut ([\matrix-org#9645](matrix-org#9645)). Fixes element-hq/element-web#18497 and element-hq/element-web#18964. * Fix search not being cleared when clicking on a result ([\matrix-org#9635](matrix-org#9635)). Fixes element-hq/element-web#23845. * Fix screensharing in 1:1 calls ([\matrix-org#9612](matrix-org#9612)). Fixes element-hq/element-web#23808. * Fix the background color flashing when joining a call ([\matrix-org#9640](matrix-org#9640)). * Fix the size of the 'Private space' icon ([\matrix-org#9638](matrix-org#9638)). * Fix reply editing in rich text editor (https ([\matrix-org#9615](matrix-org#9615)). * Fix thread list jumping back down while scrolling ([\matrix-org#9606](matrix-org#9606)). Fixes element-hq/element-web#23727. * Fix regression with TimelinePanel props updates not taking effect ([\matrix-org#9608](matrix-org#9608)). Fixes element-hq/element-web#23794. * Fix form tooltip positioning ([\matrix-org#9598](matrix-org#9598)). Fixes element-hq/element-web#22861. * Extract Search handling from RoomView into its own Component ([\matrix-org#9574](matrix-org#9574)). Fixes element-hq/element-web#498. * Fix call splitbrains when switching between rooms ([\matrix-org#9692](matrix-org#9692)). * Fix replies to emotes not showing as inline ([\matrix-org#9707](matrix-org#9707)). Fixes element-hq/element-web#23903.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As a gut reaction, the new replies are a lot harder for me to follow what's going on and pick them out.
Full-size:
It looks like this was updated in matrix-org/matrix-react-sdk#3553 and has finally made it to https://develop.element.io where I noticed it.
Reading through the PR, @nadonomy's comment about "The avatars in replies were useful, so would prefer to keep them" made me realize that this might be the big missing piece for me. Although the discussion for keeping them was leaned towards making it like small pill style and the big left spacing would still be gone 🤷
In terms of spacing and colors, it's probably more something to get used to. Although previously the bottom margin on the quote made it easier to distinguish the actual message from the quote.
This issue seems like it may be more personal feedback than an actionable issue. Feel free to close 🙇
The text was updated successfully, but these errors were encountered: