Panzoom: Use dimensões adequadas para contenção

Criado em 30 ago. 2016  ·  10Comentários  ·  Fonte: timmywil/panzoom

Assunto do problema

A contenção está usando as dimensões pai no lugar das dimensões do elemento panorâmico às vezes.

Seu ambiente

  • versão de jquery.panzoom 3.2.2
  • versão do jquery 3.1.0
  • qual navegador e sua versão Chrome 52

    Passos para reproduzir

http://jsbin.com/cidebod/edit?html , css, js, output

bug

Comentários muito úteis

@arkoe atualizou a solicitação pull # 279 para lidar com seu problema. Infelizmente, não pude atualizar seus exemplos. Me avise se funcionar.

Todos 10 comentários

@timmywil , tivemos esse bug recentemente e acabamos de escrever uma correção e abrir um PR. Verifique: https://github.com/timmywil/jquery.panzoom/pull/279/files ou baixe diretamente .

@awartani Estou enfrentando o mesmo problema e sua correção não funcionou.

O problema ocorre quando o div pai tem dimensões menores do que o elemento panzoom (a imagem).

@timmywil Alguma ideia de quando isso será consertado? Eu realmente amo o projeto e adoraria usá-lo :-).

@arkoe Acho que você testou o branch errado, atualizei seu exemplo. Verifique http://jsbin.com/gigadis/edit?html , css, js, output

@awartani Em seu exemplo, o elemento panzoom (quadrado azul) é menor que o pai. Quando você torna o elemento maior que seu pai, ele apresenta erros. Observe também que não fui eu que criei esse problema; Estou apenas enganchando :)

@Arkoe , presumi que você criou esse problema. Agora entendo o problema e acho que pode ser fácil de resolver. Por favor, dê uma olhada no Pull Request # 279 especificamente:

if (contain === 'invert' || (contain === 'automatic' && zoomAspectH < 1.01)) {
//original code
} else {
//my fix
var originalHeight = height / scale;
var originDiff = (originalHeight - height) / 2;
matrix[5] = Math.max(Math.min(matrix[5], conHeight - originalHeight + originDiff), -originDiff);
}

Como não sei muito como testar a primeira instrução if e não é o que estou procurando, consertei a instrução else . Eu acredito que se você pegar a mesma correção e ajustá-la um pouco, você será capaz de resolvê-la.

@awartani , tenho usado suas relações públicas e tentei brincar com

@arkoe, eu tinha algo para tentar, mas preciso entrar e tentar, você pode enviar um código de amostra e tentarei dar uma olhada nesta noite.

@awartani Eu @timmywil , demo2.html está usando sua correção. http://arko.io/x/panzoom/demo.html http://arko.io/x/panzoom/demo2.html

@arkoe atualizou a solicitação pull # 279 para lidar com seu problema. Infelizmente, não pude atualizar seus exemplos. Me avise se funcionar.

Thnx @awartani. Corrigiu meus problemas perfeitamente!

Esta página foi útil?
0 / 5 - 0 avaliações