添加缩略图背景 #7
Merged
添加缩略图背景 #7
Conversation
This file contains 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
|
我稍微看了看, 挺好的, 能够大幅度提升用户体验, 就是交互上还略粗糙, 所以我加上了测试字样, 并默认不开启. 主要是在加载上的交互还有欠缺考虑, 比如没有loading提示, 以及加载失败后的提示, 原先的被覆盖了. |
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.
添加在默认情况下。在网络环境较差，无法立即获取到远程图片的情况下，以缩略图形式提供给用户一个大概的浏览（即将缩略图放大）