增加缩略图预览 #217
Merged
增加缩略图预览 #217
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
单图时，图片会溢出内容区域。 |
单图显示问题， 我稍晚改下。缩略图预览的问题应该是之前数据库里面的图片不存在缩略图，所以第一次拉取缩略图失败后再拉取原图的过程中，可能导致的闪烁，现在应该有两种处理方式：1.数据库增加一个字段，表示当前记录是否存在缩略图2.统一对过往图片进行压缩处理 |
期待优化后的处理方式。 |
|
S3的压缩流程是什么 |
现在还没有添加S3的压缩，只添加了本地的上传接口的图片压缩功能 |
xuewenG
pushed a commit
to xuewenG/moments
that referenced
this pull request
Dec 6, 2024
目前有个问题，就是远程图片也加载缩略图，但是不存在。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
首页列表在展示多条记录时，如果存在多个高品质原图，在带宽有限时会影响加载速度。增加图片上传后，生成一张压缩图片，点击展开后加载原图，