Merge remote-tracking branch 'upstream/master'
This commit is contained in:
@@ -66,6 +66,10 @@ public class BlurhashCrossfadeDrawable extends Drawable{
|
||||
|
||||
public void setImageDrawable(Drawable imageDrawable){
|
||||
this.imageDrawable=imageDrawable;
|
||||
if(imageDrawable!=null){
|
||||
width=imageDrawable.getIntrinsicWidth();
|
||||
height=imageDrawable.getIntrinsicHeight();
|
||||
}
|
||||
invalidateSelf();
|
||||
}
|
||||
|
||||
@@ -99,11 +103,15 @@ public class BlurhashCrossfadeDrawable extends Drawable{
|
||||
|
||||
@Override
|
||||
public int getIntrinsicWidth(){
|
||||
if(width==0)
|
||||
return imageDrawable==null ? 1920 : imageDrawable.getIntrinsicWidth();
|
||||
return width;
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getIntrinsicHeight(){
|
||||
if(height==0)
|
||||
return imageDrawable==null ? 1080 : imageDrawable.getIntrinsicHeight();
|
||||
return height;
|
||||
}
|
||||
|
||||
|
||||
@@ -734,9 +734,18 @@ public class PhotoViewer implements ZoomPanView.Listener{
|
||||
public void onBind(Attachment item){
|
||||
super.onBind(item);
|
||||
FrameLayout.LayoutParams params=(FrameLayout.LayoutParams) imageView.getLayoutParams();
|
||||
params.width=item.getWidth();
|
||||
params.height=item.getHeight();
|
||||
ViewImageLoader.load(this, listener.getPhotoViewCurrentDrawable(getAbsoluteAdapterPosition()), new UrlImageLoaderRequest(item.url), false);
|
||||
Drawable currentDrawable=listener.getPhotoViewCurrentDrawable(getAbsoluteAdapterPosition());
|
||||
if(item.hasKnownDimensions()){
|
||||
params.width=item.getWidth();
|
||||
params.height=item.getHeight();
|
||||
}else if(currentDrawable!=null){
|
||||
params.width=currentDrawable.getIntrinsicWidth();
|
||||
params.height=currentDrawable.getIntrinsicHeight();
|
||||
}else{
|
||||
params.width=1920;
|
||||
params.height=1080;
|
||||
}
|
||||
ViewImageLoader.load(this, currentDrawable, new UrlImageLoaderRequest(item.url), false);
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -778,9 +787,18 @@ public class PhotoViewer implements ZoomPanView.Listener{
|
||||
super.onBind(item);
|
||||
playerReady=false;
|
||||
FrameLayout.LayoutParams params=(FrameLayout.LayoutParams) wrap.getLayoutParams();
|
||||
params.width=item.getWidth();
|
||||
params.height=item.getHeight();
|
||||
wrap.setBackground(listener.getPhotoViewCurrentDrawable(getAbsoluteAdapterPosition()));
|
||||
Drawable currentDrawable=listener.getPhotoViewCurrentDrawable(getAbsoluteAdapterPosition());
|
||||
if(item.hasKnownDimensions()){
|
||||
params.width=item.getWidth();
|
||||
params.height=item.getHeight();
|
||||
}else if(currentDrawable!=null){
|
||||
params.width=currentDrawable.getIntrinsicWidth();
|
||||
params.height=currentDrawable.getIntrinsicHeight();
|
||||
}else{
|
||||
params.width=1920;
|
||||
params.height=1080;
|
||||
}
|
||||
wrap.setBackground(currentDrawable);
|
||||
progressBar.setVisibility(item.type==Attachment.Type.VIDEO ? View.VISIBLE : View.GONE);
|
||||
if(itemView.isAttachedToWindow()){
|
||||
reset();
|
||||
@@ -841,7 +859,9 @@ public class PhotoViewer implements ZoomPanView.Listener{
|
||||
@Override
|
||||
public boolean onError(MediaPlayer mp, int what, int extra){
|
||||
Log.e(TAG, "video player onError() called with: mp = ["+mp+"], what = ["+what+"], extra = ["+extra+"]");
|
||||
return false;
|
||||
Toast.makeText(activity, R.string.error_playing_video, Toast.LENGTH_SHORT).show();
|
||||
onStartSwipeToDismissTransition(0f);
|
||||
return true;
|
||||
}
|
||||
|
||||
public void prepareAndStartPlayer(){
|
||||
@@ -862,6 +882,8 @@ public class PhotoViewer implements ZoomPanView.Listener{
|
||||
player.prepareAsync();
|
||||
}catch(IOException x){
|
||||
Log.w(TAG, "Error initializing gif player", x);
|
||||
Toast.makeText(activity, R.string.error_playing_video, Toast.LENGTH_SHORT).show();
|
||||
onStartSwipeToDismissTransition(0f);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -119,8 +119,10 @@ public class ZoomPanView extends FrameLayout implements ScaleGestureDetector.OnS
|
||||
|
||||
int width=right-left;
|
||||
int height=bottom-top;
|
||||
if(width==0 || height==0)
|
||||
if(width==0 || height==0 || child.getWidth()==0 || child.getWidth()==0){
|
||||
matrix.reset();
|
||||
return;
|
||||
}
|
||||
|
||||
float scale=Math.min(width/(float)child.getWidth(), height/(float)child.getHeight());
|
||||
minScale=scale;
|
||||
|
||||
@@ -28,6 +28,7 @@ public class MediaAttachmentViewController{
|
||||
private final Context context;
|
||||
private boolean didClear;
|
||||
private Status status;
|
||||
private Attachment attachment;
|
||||
|
||||
public MediaAttachmentViewController(Context context, MediaGridStatusDisplayItem.GridItemType type){
|
||||
view=context.getSystemService(LayoutInflater.class).inflate(switch(type){
|
||||
@@ -54,6 +55,7 @@ public class MediaAttachmentViewController{
|
||||
|
||||
public void bind(Attachment attachment, Status status){
|
||||
this.status=status;
|
||||
this.attachment=attachment;
|
||||
crossfadeDrawable.setSize(attachment.getWidth(), attachment.getHeight());
|
||||
crossfadeDrawable.setBlurhashDrawable(attachment.blurhashPlaceholder);
|
||||
crossfadeDrawable.setCrossfadeAlpha(0f);
|
||||
@@ -76,6 +78,11 @@ public class MediaAttachmentViewController{
|
||||
crossfadeDrawable.setImageDrawable(drawable);
|
||||
if(didClear)
|
||||
crossfadeDrawable.animateAlpha(0f);
|
||||
// Make sure the image is not stretched if the server returned wrong dimensions
|
||||
if(drawable!=null && (drawable.getIntrinsicWidth()!=attachment.getWidth() || drawable.getIntrinsicHeight()!=attachment.getHeight())){
|
||||
photo.setImageDrawable(null);
|
||||
photo.setImageDrawable(crossfadeDrawable);
|
||||
}
|
||||
}
|
||||
|
||||
public void clearImage(){
|
||||
|
||||
Reference in New Issue
Block a user