Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unoccupied empty column reserved in the message screen? #26282

Closed
nabijaczleweli opened this issue Oct 2, 2023 · 4 comments
Closed

Unoccupied empty column reserved in the message screen? #26282

nabijaczleweli opened this issue Oct 2, 2023 · 4 comments
Labels
T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter

Comments

@nabijaczleweli
Copy link

nabijaczleweli commented Oct 2, 2023

Steps to reproduce

  1. open a room

Outcome

What did you expect?

Messages are shown in the part of the window that isn't the sidebar

What happened instead?

There's a sidebar on the right taht does nothing:
image

Fix

This appears to be an uhhhhhhhhhhhhh this thing

Which is empty. So unchecking the width: 90px; statement for .mx_EventTile[data-layout=group] .mx_EventTile_msgOption, .mx_EventTile[data-layout=irc] .mx_EventTile_msgOption { makes the page render correctly:
image

Well, "correctly". At least it's just the padding: var(--RoomView_MessageList-padding); on .mx_RoomView_MessageList like in #26281.

Indeed, replacing that clause with

    padding: 0;
    padding-left: calc(var(--RoomView_MessageList-padding)/2);

turns it into something that isn't an affront to all screen users:

Operating system

Windows

Application version

Element version: 1.11.45 Olm version: 3.2.14

How did you install the app?

clicked enter on "Element Setup.exe"

Homeserver

nabijaczleweli.xyz Synapse 1.93.0-1

Will you send logs?

No

@weeman1337
Copy link
Contributor

@nabijaczleweli Thank you for opening an issue. Unfortunately there is not enough information there for me to be able to reproduce it. Please update the description with steps/screenshots/video/more details so our developers can have a look at it: Can you send the exact settings from the appearance section?

@weeman1337 weeman1337 added the X-Needs-Info This issue is blocked awaiting information from the reporter label Oct 9, 2023
@nabijaczleweli
Copy link
Author

Unclear to me how many more screenshots you can possibly want, but sure. Browser:
image
Desktop:
image

Browser appearance:
image
Desktop appearance:
image
image

@t3chguy
Copy link
Member

t3chguy commented Oct 12, 2023

Looks like you mean the read receipts column?

image

You can disable it. .mx_EventTile_msgOption is controlled by Show read receipts sent by other users

@t3chguy t3chguy closed this as completed Oct 12, 2023
@nabijaczleweli
Copy link
Author

It sure appears to be. But it's even more egregious in your screenshot, why is it so wide? It's not that wide on a phone (otherwise that'd be half your screen)? The empty column has no bearing on the actual receipts (as proven by "if you set it to width: 0 they still show, and the actual post on which they are shrinks by how wide they are"). The receipts aren't actually that wide? Even in your screenshot? Why is significant hunk of the screen designed to be completely empty?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter
Projects
None yet
Development

No branches or pull requests

3 participants