element-hq / element-hq/element-web

24hr timestamps should have shorter width than 12hr timestamps [irc theme]

Open
#17,871 2 comments 0 reactions 0 assignees View on GitHub
A-Appearance A-IRC-Layout T-Enhancement
Dominant language
TypeScript
Stars
13.5k
Forks
2.8k
PR merge metrics
PR metrics pending

Description

#### Description

24hr timestamps are shorter than 12hr timestamps and therefore leave too much blank space.

`.mx_IRCLayout .mx_EventTile > a { min-width: 45px; }` and `.mx_IRCLayout .mx_EventTile .mx_MessageTimestamp { width: 45px; }` work well for 12hr timestamps (which vary in length), but 24hr timestamps, which dont have AM/PM and do enforce leading zeros(more consistent width), would be better off shorter, suggesting 30px or 35px.

12hr timestamps with a 45px width:
![Screenshot_20210703_151944](https://user-images.githubusercontent.com/41446523/124364913-ab83dd80-dc12-11eb-9a45-80c8760a0179.png)

24hr timestamps with a 45px width:
![Screenshot_20210703_151955](https://user-images.githubusercontent.com/41446523/124364920-bd658080-dc12-11eb-8657-14298209b57b.png)

24hr timestamps with a 30px width (set by a theming userscript):
![Screenshot_20210703_152010](https://user-images.githubusercontent.com/41446523/124364923-c6eee880-dc12-11eb-9dc1-932c5537c1dc.png)

24hr timestamps with a 35px width (set by a theming userscript) (note: profile name width change unrelated):
![Screenshot_20210703_152905](https://user-images.githubusercontent.com/41446523/124365078-b68b3d80-dc13-11eb-902c-b07346d6e3b7.png)

#### Version information

- **Platform**: web (in-browser)

For the web app:

- **Browser**: Firefox 88.0 64bit
- **OS**: Fedora Linux
- **URL**: develop.element.io

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.