Ticket #706 (closed バグ指摘: 修正済)

Opened 13 years ago

Last modified 12 years ago

拡大画像の表示の jQuery 化

Reported by: Seasoft Owned by: nanasess
Priority: Milestone: EC-CUBE2.11.2
Component: フロント Version: 2.5-dev
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

Fancybox で一旦実装したが、IE6, IE7 で不具合があるため、Facebox に置換する。

Attachments

iPhone.jpg Download (73.0 KB) - added by nanasess 12 years ago.
Facebox ではブラウザの画面サイズに合わせて画像が縮小されない
facebox_ie8.GIF Download (69.7 KB) - added by habu 12 years ago.
IE8では、画像がFaceBoxの枠からはみ出る

Change History

comment:1 follow-up: ↓ 2 Changed 13 years ago by Seasoft

  • Description modified (diff)

EC-CUBE コミュニティ版の利用者様から下記の不具合報告をいただきました。

それと昨夜弄っていて、一か所”不具合(?)”を見つけたのでご報告です。

jquery.fancybox.cssの最初の

html, body {
	height: 100%;
}

があると、bodyがモニターサイズ(?)の高さになってしまい、bodyに背景付けた時にきちんと表示されません。
なのでデフォでコメントアウトしておくのがいいと思いますが・・・

最新の Fancybox 1.3.1 では、上記の記述は削除されている模様。 Fancybox のバージョンアップで対応を検討。

comment:2 in reply to: ↑ 1 ; follow-up: ↓ 3 Changed 13 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • Status changed from new to assigned

r18614 で対応。

comment:3 in reply to: ↑ 2 Changed 13 years ago by Seasoft

r18615 でコミット漏れを対応。

comment:4 Changed 13 years ago by Seasoft

  • Status changed from assigned to closed
  • Resolution set to 修正済

comment:5 Changed 13 years ago by nanasess

  • Status changed from closed to reopened
  • Resolution 修正済 deleted

正式版へ未反映ですので差し戻します

comment:6 Changed 13 years ago by kajiwara

  • Status changed from reopened to closed
  • Resolution set to 修正済

version-2_5-dev ブランチで対応済みですので一旦 close します

comment:7 follow-up: ↓ 8 Changed 13 years ago by Seasoft

  • Status changed from closed to reopened
  • Version changed from コミュニティ (eccube-comu) to 2.5-dev
  • Resolution 修正済 deleted
  • Milestone changed from EC-CUBE2.5.0alpha to EC-CUBE2.5.0beta

IE で、関連画像の読み込み不良が発生。操作性も悪くなっている。原因は CSS の AlphaImageLoader? のパスが、表示中のURLを基準とするため。(通常は、CSS 自身を基準とする。)

下記の対処方法が考えられる。

  • CSS を絶対パスで出力する仕組みを用意する。
  • /html/user_data/packages/default/jquery.fancybox/ を /html/fancybox/ へ複製する。
  • 別のライブラリを使用する。

comment:8 in reply to: ↑ 7 Changed 12 years ago by Seasoft

IE で、関連画像の読み込み不良が発生。操作性も悪くなっている。原因は CSS の AlphaImageLoader?? のパスが、表示中のURLを基準とするため。(通常は、CSS 自身を基準とする。)

IE8 では、問題無いことを確認。

致命的な状況ではないため、特段の問題提起が無いようでしたら、本件は黙視し、マイルストーンを EC-CUBE2.5.0 に変更します。

  • 別のライブラリを使用する。

IE6 対応の prettyPhoto に変更する場合の例。
 http://seasoft.jp1.cx/ec/tips/prettyphoto

comment:9 follow-up: ↓ 11 Changed 12 years ago by Seasoft

フォーラムにて、pigmon 様より IE7 での NG 報告あり。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=4708&forum=5&post_id=29106

comment:10 Changed 12 years ago by Seasoft

  • Description modified (diff)

comment:11 in reply to: ↑ 9 ; follow-up: ↓ 12 Changed 12 years ago by Seasoft

Seasoft への返信

フォーラムにて、pigmon 様より IE7 での NG 報告あり。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=4708&forum=5&post_id=29106

Fancybox への移行とします。

comment:12 in reply to: ↑ 11 Changed 12 years ago by Seasoft

Seasoft への返信

Seasoft への返信

フォーラムにて、pigmon 様より IE7 での NG 報告あり。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=4708&forum=5&post_id=29106

Fancybox への移行とします。

正しくは Facebox でした。

comment:13 Changed 12 years ago by Seasoft

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.0

Changed 12 years ago by nanasess

Facebox ではブラウザの画面サイズに合わせて画像が縮小されない

comment:14 Changed 12 years ago by nanasess

Seasoft への返信

正しくは Facebox でした。

XCode 付属の iOSシミュレータ バージョン 4.2 (235) で確認したところ, Facebox では, ブラウザのサイズに合わせて拡大画像が縮小されません. Facebox ではブラウザの画面サイズに合わせて画像が縮小されない

Fancybox はうまく表示されますので, スマートフォン用のテンプレートは Fancybox を使用した方が良いかもしれません.

comment:15 follow-up: ↓ 16 Changed 12 years ago by nanasess

スマートフォン用テンプレートは r20014 で fancybox に戻しました

comment:16 in reply to: ↑ 15 ; follow-up: ↓ 17 Changed 12 years ago by Seasoft

nanasess への返信

スマートフォン用テンプレートは r20014 で fancybox に戻しました

Android 2.3 HVGA 画面ですと、拡大画像の方が元の画像よりも小さく表示されます。画像へ直リンクのほうが使い勝手よいかも。

comment:17 in reply to: ↑ 16 Changed 12 years ago by Seasoft

r20019 で、商品詳細画面のみスマホは対象から外しました。

多分、他の画面は大丈夫だと推測しています。

comment:18 Changed 12 years ago by Seasoft

  • Status changed from reopened to closed
  • Resolution set to 修正済

Changed 12 years ago by habu

IE8では、画像がFaceBoxの枠からはみ出る

comment:19 follow-ups: ↓ 20 ↓ 22 Changed 12 years ago by habu

  • Status changed from closed to reopened
  • Resolution 修正済 deleted

IE8では、画像が枠からはみ出します。 IE8では、画像がFaceBoxの枠からはみ出る Firefox 3.6.15と、Chrome 10.0.648.127では正常に表示されました。

comment:20 in reply to: ↑ 19 Changed 12 years ago by Seasoft

下記環境で再現しませんでした。

OS 名	Microsoft Windows 7 Home Premium
バージョン	6.1.7600 ビルド 7600
システムの種類	x64-ベース PC
IE 8.0.7600.16385 更新バージョン 0

発現条件を特定したいので、詳細な情報の提供をお願いします。

また、他の環境の IE での発現の有無をお知らせいただけると助かります。> 皆様

なお、環境依存が強いようでしたら、別チケットでの継続対応とする予定です。

comment:21 Changed 12 years ago by kotani

下記環境で再現しませんでした。

OS 名	Microsoft Windows 7 Professional
バージョン	6.1 ビルド 7600
IE 8.0.7600.16385 更新バージョン 0

comment:22 in reply to: ↑ 19 Changed 12 years ago by kotani

habu への返信

IE8では、画像が枠からはみ出します。 IE8では、画像がFaceBoxの枠からはみ出る Firefox 3.6.15と、Chrome 10.0.648.127では正常に表示されました。

再現しないためクローズします。

comment:23 Changed 12 years ago by kotani

  • Status changed from reopened to closed
  • Resolution set to 修正済

comment:24 follow-up: ↓ 26 Changed 12 years ago by nanasess

  • Status changed from closed to reopened
  • 修正済み unset
  • Type changed from 新規開発 to バグ指摘
  • Resolution 修正済 deleted
  • Milestone changed from EC-CUBE2.11.0 to EC-CUBE2.11.2

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=7962&forum=10&post_id=38470

IE7 で画像が枠からはみ出すようですので差し戻します

comment:25 Changed 12 years ago by nanasess

  • Owner changed from Seasoft to nanasess
  • Status changed from reopened to new

comment:26 in reply to: ↑ 24 ; follow-up: ↓ 27 Changed 12 years ago by nanasess

  • Status changed from new to assigned
  • 修正済み set

nanasess への返信

IE7 で画像が枠からはみ出すようですので差し戻します

r20917 修正してみました

comment:27 in reply to: ↑ 26 Changed 12 years ago by Seasoft

nanasess への返信

r20917 修正してみました

IETest (on WinXP) IE7 で改善を確認しました。

comment:28 Changed 12 years ago by kajiwara

あらゆる環境でチェックはできておりませんが、現状は問題ないようです。

一応、2.11.2リリースギリギリまで調査しようと思いますが、特に問題ないようでしたら、そのままこちらの内容で完了とします。

comment:29 Changed 12 years ago by kajiwara

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.