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

Switching to a long chat makes browser lag #33

Closed
tonowoe opened this issue Nov 24, 2018 · 2 comments
Closed

Switching to a long chat makes browser lag #33

tonowoe opened this issue Nov 24, 2018 · 2 comments
Labels
bug High Priority Performance
Projects
Epic - UI/UX Improvements

Comments

@tonowoe
Copy link

tonowoe commented Nov 24, 2018

If there's a chat which chat log is fairly long, activating the chat will lag the browser for a second. Maybe show only last 50 messages and show the rest if scrolled up or something?
@NullVoxPopuli
Copy link
Owner

Most definitely. In the latest commit, I I actually disabled the render the scene, not to world technique I was using, so I could figure out scroll issues.

Adding it back, or making my own will go into the next release.

Thanks for reporting!

mdtanrikulu pushed a commit to mdtanrikulu/emberclear that referenced this issue Feb 15, 2019
@NullVoxPopuli
unread counting, read timestamps, read marking. Resolves NullVoxPopul…
0facbde 
…i#33. Todo: NullVoxPopuli#39, and…
mdtanrikulu pushed a commit to mdtanrikulu/emberclear that referenced this issue Feb 15, 2019
@NullVoxPopuli
Merge branch 'unread-counts' into 'master'
42757fa 
unread counting, read timestamps, read marking. Resolves NullVoxPopuli#33. Todo: NullVoxPopuli#39, and…

Closes NullVoxPopuli#33

See merge request NullVoxPopuli/emberclear!61
@NullVoxPopuli NullVoxPopuli removed this from To do in General Project Board Feb 19, 2019
@NullVoxPopuli
Copy link
Owner

I believe this is resolved now -- at most, there should only be 10 messages rendered off screen (5 above, and 5 below)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug High Priority Performance
Projects
No open projects
Epic - UI/UX Improvements
  
To do
Milestone
No milestone
Development

No branches or pull requests
2 participants
@NullVoxPopuli @tonowoe