12h timestamp collides with encryption icon

[turt2live]

Description

Log: not sent

Version information

• Platform: web (in-browser)
• Browser: Chrome 58
• OS: Windows 10
• URL: riot.im/develop

[t3chguy]

where mah mention!?

[turt2live]

@t3chguy

[t3chguy]

ty

[ara4n]

looks like this is fixed now?

[t3chguy]

still broken for me on /develop/

[t3chguy]

@ara4n

[t3chguy]

better eg

[lampholder]

We shouldn't push this live with the overlap glitch, but we shouldn't block the release on this feature... Hopefully the fix is small enough that it's easy to roll in with whatever else we're doing.

[turt2live]

I tried some CSS in Chrome to make this a bit nicer, and can't really decide on a fix. Input appreciated:

1. Pushing the timestamp 8px to the left (colliding with the hover animation)
   
2. Making the sender profile larger by ~10px
   
   The downside being 24hr time looks a bit weird
   
3. Something I haven't considered

[uhoreg]

How about removing the space before the AM/PM? Or making the font size slightly smaller?

[turt2live]

removing the space doesn't save enough space:

Doing both font size and removing the space seems to make it fit (11px -> 10px)

Edit: And it doesn't look horrible for 24h time. Patch incoming

[turt2live]

fixed via matrix-org/matrix-react-sdk#969 and #4085

Ended up going with the smaller text and no space solution.