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

升级到 Glide 4.2+ #93

Open
zhanghai opened this issue Apr 17, 2018 · 0 comments
Open

升级到 Glide 4.2+ #93

zhanghai opened this issue Apr 17, 2018 · 0 comments
Assignees
Labels

Comments

@zhanghai
Copy link
Owner

zhanghai commented Apr 17, 2018

Glide 4.2.0 相对于 4.1.1 引入了多个问题:

  1. 默认的 DownsamplingStrategy 由 AT_LEAST 变更为 FIT_OUTSIDE,导致部分长图(正确地)过大而无法加载。
  2. 在每张图第一次 SharedElementTransition 时大概率闪过白屏,但之后不会,推测与 cache 相关,未知是否与新的 ViewTarget getSize() 实现有关。如果回退 1 为 AT_LEAST,bug 变为 transition 时部分稍大图片出现由小变大动画。
4.7.1

getSize(): 104x104
onResourceReady(): 104x104
getSize(): 0x0
onMeasure(): 1080x1439
getSize(): 1080x1439
onResourceReady(): 104x104
onResourceReady(): 1080x1440
onMeasure(): 1080x1439

onResourceReady(): 104x104
getSize(): 0x0
onMeasure(): 1080x1439
getSize(): 1080x1439
onResourceReady(): 1080x1440
onMeasure(): 1080x1439
onResourceReady(): 104x104

4.1.1

onMeasure(): 1080x1439
getSize(): 104x104
setResource(): 160x160
getSize(): 1080x1439
setResource(): 540x720
onMeasure(): 1080x1439
setResource(): 160x160

4.7.1 with AT_LEAST

getSize(): 104x104
onResourceReady(): 104x104
getSize(): 0x0
onMeasure(): 1080x1439
getSize(): 1080x1439
onResourceReady(): 104x104
onResourceReady(): 540x720
onMeasure(): 1080x1439

getSize(): 104x104
onResourceReady(): 104x104
getSize(): 0x0
onMeasure(): 1080x1439
getSize(): 1080x1439
onResourceReady(): 540x720
onMeasure(): 1080x1439
onResourceReady(): 104x104

目前已回退到 4.1.1。

之后有时间 bisect 吧。

@zhanghai zhanghai added the bug label Apr 17, 2018
@zhanghai zhanghai self-assigned this Apr 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant