Improve readability of RegularMessageItem widget #41

Merged

emersion merged 4 commits from bl1nk/goguma:push-pltrntyxmmtp into master 2025-10-05 22:34:33 +02:00

Conversation 0 Commits 4 Files changed 1 +33 -27

bl1nk commented 2025-10-04 16:18:05 +02:00

Contributor

Copy link

Hey there,

first contribution here, please let me know if I missed something.

We talked about it in the IRC channel but to summarize: With this PR I would like to update the look of message bubbles when not using the compact style. It improves readability and aligns Goguma a bit with other chat apps out there.

The changes in this PR are:

• Change message bubble colors. Use the primary theme color for the background of outgoing messages. Use a light color for the background of incoming messages. Reuse the former bubble color as the color of sender nicknames.
• Remove the nickname on outgoing messages.
• Limit message bubble widths to 80% of the screen width at most. Most other chat apps do this as well do keep the left-right alignment of incoming/outgoing messages visible.

Check the screenshots for a before and after in light and dark modes:

Before	After

Happy to make any changes based on review feedback.

Hey there, first contribution here, please let me know if I missed something. We talked about it in the IRC channel but to summarize: With this PR I would like to update the look of message bubbles when not using the compact style. It improves readability and aligns Goguma a bit with other chat apps out there. The changes in this PR are: - Change message bubble colors. Use the primary theme color for the background of outgoing messages. Use a light color for the background of incoming messages. Reuse the former bubble color as the color of sender nicknames. - Remove the nickname on outgoing messages. - Limit message bubble widths to 80% of the screen width at most. Most other chat apps do this as well do keep the left-right alignment of incoming/outgoing messages visible. Check the screenshots for a before and after in light and dark modes: | Before | After | |---------|---------| |  |  | |  |  | Happy to make any changes based on review feedback.

dark-after.png

247 KiB

dark-before.png

230 KiB

light-after.png

252 KiB

light-before.png

236 KiB

👍 1 🎉 1

bl1nk force-pushed push-pltrntyxmmtp from 888f8511f9 to 236d1f6e21 2025-10-04 16:26:32 +02:00 Compare

bl1nk force-pushed push-pltrntyxmmtp from 236d1f6e21 to 9ec16a967c 2025-10-04 16:47:24 +02:00 Compare

bl1nk added 1 commit 2025-10-04 17:08:56 +02:00

Set textColor using colorScheme ... 05a5f3c195

Uses onPrimaryContainer for the text color of outgoing messages, fitting the background color of primaryContainer.

Uses onSurface for incoming messages, fitting the background color of surfaceContainer.

emersion approved these changes 2025-10-05 22:32:30 +02:00

emersion left a comment

Owner

Copy link

Very nice! Thanks so much for working on this!

Very nice! Thanks so much for working on this!

emersion merged commit 05a5f3c195 into master 2025-10-05 22:34:33 +02:00

bl1nk referenced this pull request 2025-10-08 23:05:07 +02:00

Fix low contrast on certain sender colors #43

emersion referenced this pull request 2025-12-27 18:17:15 +01:00

Improve message view appearance #120

Sign in to join this conversation.

Reviewers

No reviewers

emersion

Labels

Clear labels   

bug

  

enhancement

  

good first issue

No labels

bug

enhancement

good first issue

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

emersion/goguma!41

No description provided.