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

feat(webui): virtual message list #215

Merged
merged 3 commits into from
Apr 9, 2021
Merged

feat(webui): virtual message list #215

merged 3 commits into from
Apr 9, 2021

Conversation

shigma
Copy link
Member

@shigma shigma commented Apr 9, 2021

支持虚拟的消息列表。

  • 同屏只渲染固定数量的消息,其他消息以 padding 的形式留空
  • 存储并估计消息的高度,解决空屏问题
  • 修复虚拟列表带来的 quote 点击失效问题
  • 实现在适当的时机 scrollToBottom 的行为

@shigma shigma self-assigned this Apr 9, 2021
@shigma shigma changed the base branch from master to develop April 9, 2021 04:32
@codecov
Copy link

codecov bot commented Apr 9, 2021

Codecov Report

Merging #215 (cef7e96) into develop (b429522) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #215   +/-   ##
========================================
  Coverage    92.34%   92.34%           
========================================
  Files           53       53           
  Lines        10638    10638           
  Branches      2176     2176           
========================================
  Hits          9824     9824           
  Misses         814      814           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b429522...cef7e96. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2021

This pull request introduces 11 alerts when merging 06cb17d into cfe3209 - view on LGTM.com

new alerts:

  • 11 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2021

This pull request introduces 10 alerts when merging 06cb17d into b429522 - view on LGTM.com

new alerts:

  • 10 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2021

This pull request introduces 10 alerts when merging 5267a8a into b429522 - view on LGTM.com

new alerts:

  • 10 for Unused variable, import, function or class

@shigma shigma merged commit 3a72c99 into develop Apr 9, 2021
@shigma shigma deleted the virtual-list branch April 9, 2021 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant