Panzoom: 封じ込めには適切な寸法を使用してください

作成日 2016年08月30日  ·  10コメント  ·  ソース: timmywil/panzoom

問題の主題

封じ込めでは、パンズーム要素のディメンションの代わりに親ディメンションが使用されることがあります。

あなたの環境

  • jquery.panzoom3.2.2のバージョン
  • jquery3.1.0のバージョン
  • どのブラウザとそのバージョンChrome52

    再現する手順

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

bug

最も参考になるコメント

@arkoe問題を処理するためにプルリクエスト#279を更新しました。 残念ながら、私はあなたの例を更新することができませんでした。 それが機能するかどうか私に知らせてください。

全てのコメント10件

@timmywil最近このバグが発生し、修正を書き込んでPRを開きました。 チェックアウトしてください: https直接ダウンロードしてください

@awartani私は同じ問題に直面していて、あなたの修正は機能しませんでした。

この問題は、親divのサイズがpanzoom要素(画像)よりも小さい場合に発生します。

@timmywilこれがいつ修正されるかについて何か考えはありますか? 私はこのプロジェクトが大好きで、それを使いたいと思っています:-)。

@arkoe間違ったブランチをテストしたと思いますが、例を更新しました。 http://jsbin.com/gigadis/edit?html、css、js、outputを確認してください

@awartaniあなたの例では、panzoom要素(青い四角)は親よりも小さいです。 要素をその親よりも大きくすると、バグが発生します。 また、私はこの問題を作成しなかったことにも注意してください。 私はただ夢中になっています:)

申し訳ありませんが@arkoeこの問題を作成したと思います。 私は今問題を理解していて、それは簡単に解決できるかもしれないと思います。 具体的にはプルリクエスト#279をご覧ください。

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);
}

最初のifステートメントをテストする方法がよくわからず、探しているものでもないため、 elseステートメントを修正しました。 同じ修正を加えて少し調整すれば、解決できると思います。

@awartani私はあなたのPRを使っていて、それで遊んでみました。 すでにかなりの時間を費やしていて、それを修正することができないようです。 これを修正する正しい方向に私を向ける可能性のある何かがあなたの心にありますか? ちなみに迅速な対応ありがとうございます! 本当に感謝。

@arkoe試してみることがありましたが、試してみる必要があります。サンプルコードをプッシュしていただければ、今夜試してみます。

@awartani 2つの例をアップロードしました。demo.htmlは@timmywilからのdistファイルでhttp://arko.io/x/panzoom/demo.html http://arko.io/x/panzoom/demo2.html

@arkoe問題を処理するためにプルリクエスト#279を更新しました。 残念ながら、私はあなたの例を更新することができませんでした。 それが機能するかどうか私に知らせてください。

Thnx @ awartani。 私の問題を完全に修正しました!

このページは役に立ちましたか?
0 / 5 - 0 評価