element-hq / element-web

A glossy Matrix collaboration client for the web.
https://element.io
GNU Affero General Public License v3.0
11.2k stars 2k forks source link

Emotes don't stand out good enough #9509

Open rkfg opened 5 years ago

rkfg commented 5 years ago

Description

Here's a proposal for better styling of emote messages. My approach is to highlight the nickname of the poster, make the message italicized and shift it a bit to the left. Here's what it would look like (I'm using a userstyle for that for a long time and I like how it looks and feels): 2019-04-17_11-59-02

Without the userstyle the first and last line (that I simply wrote by hand without /me) are indistinguishable, except the name in the first line is clickable: 2019-04-17_12-00-08

The userstyle I use:

.mx_MEmoteBody {
    margin-left: -10px;
    font-style: italic;
}

.mx_MEmoteBody_sender {
    font-weight: bold;
}

I understand that it makes italics effectively disabled in emotes as the whole message is already italicized, maybe it's better to make it gray-ish instead, it's up to you. But self-referencing should be recognized easily (to not be confused with regular messages, it's like said words vs. thoughts) and that's why I think it's important.

anoadragon453 commented 2 years ago

Perhaps instead we could simply colour the display name the same colour that is used when showing them above messages in the timeline? The colour is certainly stand out (and as a bonus will make /rainbowme commands look nicer!).

t3chguy commented 2 years ago

Ftr bubbles layout opted to have the emotes italicised too

t3chguy commented 2 years ago

Perhaps instead we could simply colour the display name the same colour that is used when showing them above messages in the timeline? The colour is certainly stand out (and as a bonus will make /rainbowme commands look nicer!).

This is an existing issue but I am failing to find it right now, also applies to state events displayed in the timeline.

turt2live commented 2 years ago

Related: https://github.com/matrix-org/matrix-react-sdk/pull/8420