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.

Sign up for GitHub

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

Jump to bottom

Message view sometimes horizontally scrolls. #41

Closed
ara4n opened this issue Aug 29, 2015 · 3 comments
Closed

Message view sometimes horizontally scrolls. #41

ara4n opened this issue Aug 29, 2015 · 3 comments
Labels
P4 [OBSOLETE LABEL] Interesting — Not yet scheduled, will accept patches

Comments

@ara4n
Copy link
Member

ara4n commented Aug 29, 2015

No description provided.

@ara4n ara4n added the ui/ux label Aug 29, 2015
@ara4n ara4n added P3 ui/ux and removed ui/ux labels Sep 7, 2015
@ara4n
Copy link
Member Author

ara4n commented Oct 11, 2015

This happens when there's some unwrappable content like a large image thumbnail that can't fit horizontally into mx_RoomView_messagePanel, whose "overflow-y: auto" causes horizontal scroll to also happen unless you explicitly disable it with overflow-x: hidden.

Given scrolling is better than hidden, we might as well leave it at low priority for now. The best fix would be for the flexbox layout to force the middle panel to push out the whole width of the browser window, but unsure how to do that right now.

@ara4n ara4n added P4 [OBSOLETE LABEL] Interesting — Not yet scheduled, will accept patches and removed P3 labels Oct 11, 2015
@ara4n ara4n modified the milestone: Ragnarok Nov 29, 2015
@ara4n
Copy link
Member Author

ara4n commented Feb 18, 2016

fixed by letting thumbnails scale horizontally.

@ara4n ara4n closed this as completed Feb 18, 2016
@ara4n
Copy link
Member Author

ara4n commented Feb 18, 2016

ffb1584

dtygel pushed a commit to coletivoEITA/riot-web that referenced this issue May 14, 2017
@MTRNord
Merge pull request element-hq#41 from RiotTranslate/weblate-riot-web-…
e5a9dd2 
…riot-web

Update from Weblate.
@linuxtim linuxtim mentioned this issue May 11, 2024
Closed
t3chguy added a commit that referenced this issue Oct 17, 2024
@t3chguy
Merge pull request #41 from element-hq/t3chguy/wat/230.1
3f67819
JohnSimonsen added a commit to verji/element-web-v2 that referenced this issue Dec 20, 2024
@JohnSimonsen
Merge pull request element-hq#41 from verji/rm/1917-Div8
96ee992 
Rm/1917 div8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P4 [OBSOLETE LABEL] Interesting — Not yet scheduled, will accept patches
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ara4n