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

Impossible to edit long messages without newlines, the editor scroll always jumps to the bottom #6825

Closed
rkfg opened this issue May 30, 2018 · 4 comments
Closed

Impossible to edit long messages without newlines, the editor scroll always jumps to the bottom #6825

rkfg opened this issue May 30, 2018 · 4 comments
Labels
P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter

Comments

@rkfg
Copy link
Contributor

rkfg commented May 30, 2018

Description

I recorded a video to better illustrate this:
bug-2018-05-30_09.43.04.zip

Steps to reproduce

  • copy&paste or write a long text without a single newline so that a vertical scrollbar appears
  • try to edit something in the beginning
  • the scroll position always resets to the bottom
  • adding a newline in the end allows for normal editing

Version information

  • Platform: web (in-browser)

For the web app:

  • Browser: Chrome 67.0.3396.18 (behaves different in Firefox, the caret still stays on top but at least it's visible)
  • OS: Debian testing amd64
  • URL: private server, 0.15.4
@rkfg rkfg mentioned this issue May 30, 2018
Closed
@lampholder
Copy link
Member

Confirmed, and enraging.

I expect this will be fixed by the inbound Slate.js rewrite of the composer.

@lampholder lampholder added T-Defect P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround type:composer labels May 30, 2018
@lampholder lampholder mentioned this issue Jun 5, 2018
Closed
@turt2live
Copy link
Member

@rkfg I can't reproduce this anymore - are you still seeing it on recent versions of Riot (0.17 or higher)?

@turt2live turt2live added the X-Needs-Info This issue is blocked awaiting information from the reporter label Oct 31, 2018
@t3chguy
Copy link
Member

t3chguy commented Mar 24, 2020

Closing as no response to unable to repro

@t3chguy t3chguy closed this as completed Mar 24, 2020
@rkfg
Copy link
Contributor Author

rkfg commented Mar 24, 2020

Sorry, didn't notice it 1.5 years ago (lol). Indeed, now it works as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@rkfg @turt2live @lampholder @t3chguy