nextcloud / nextcloud/spreed

[FOLLOW-UP] Issues and visual adjustments to chat split view

Open
#16,590 10 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

feature: chat 💬 feature: frontend 🖌️ overview
Dominant language
JavaScript
Stars
2.2k
Forks
586
Avg merge
18h 27m
Merged PRs (30d)
333

Description

How to use GitHub
  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Noticed issues:
Design follow-ups:
Before After
Image Image

Small adjustments which make the split view look more cohesive:

  • Add more spacing between messages of different authors
  • Align avatar to author name
  • reduce size of author name
  • align avatar of own messages to message container
  • Add more gap between message text and reactions row

In the inspector here are the changes I made to achieve the "After" screenshot

.wrapper.incoming .messages__avatar-wrapper {
  padding-block-start: calc(3*var(--default-grid-baseline)); }

.wrapper {
  padding: calc(2*var(--default-grid-baseline)) 0; }

.messages__author {
  font-size: small; }

.wrapper.outgoing .messages__avatar-wrapper {
  padding-block-start: calc(2*var(--default-grid-baseline)); }

.message-main--sided:has(.message-main__text):not(.message-main--compressed) .message-actions {
  padding-top: var(--default-grid-baseline); }
Message actions:
  • keep consistent placement for actions (close to top/bottom corner, regardless if inside or outside)
    • inside is covering existing reactions / timestamp
    • outside is taking reserved space from chat width
outside inside
image image
  • Consider buttons size reduction (small, 24px)

    • For both split and normal chat-view?
    • Size would be aligned with 'reply in thread' and 'react to message' buttons
    • less overlay over message content
    • less space reserved for actions, more place for content
  • Open three-dots menu with right-click

    • in addition to existing show on hover to not break flow users are used to?
  • For quick reactions, consider moving them?

Variant Mockup Pros Cons
Separate as popup Image less clicks a11y
Merge as second row Image less clicks increase expected height
Merge as same row Image less clicks won't fit on narrow screens, should be hidden
Hide under three-dots Image -- same or more clicks to find needed reactions
Pinned messages:
  • Consider another placement for the icon in split view:
image

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source files or tests are named. Start by reviewing the chat split-view implementation and the unchecked items, including the listed message, action, reaction, and pinned-message selectors. Done would require agreed design decisions and verification of the affected layouts across split and normal chat views.

Written by the indexing model from the issue text.

Assessment

Tech stack
css, javascript
Domain
design, frontend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
18/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.