XoopsX / legacy

XOOPS Cube Legacy base repository
15 stars 11 forks source link

pack2011_default theme's image slider with bug #7

Open naao opened 12 years ago

naao commented 12 years ago

pack2011_default テーマの画像スライダが、正しく動作しないです。 1枚目→2枚目はOK、 その後、2→3、3→1の時に、 3度ほど引っ掛かって戻りを繰り返したあと、 進む感じです。

xoops123 commented 12 years ago

私の環境(「どこでもXOOPS」Win、MAMP)で確認しましたが、問題ないようです。 naaoさんの環境で、ほかのJSとのバッティングとかはないでしょうか?

naao commented 12 years ago

最新の、XoopsX/legacy で動作確認した時に、クリーンインストールで2ndインストールが終了して後の、ログイン前の時点で発生しています。 もし何かのバッティングがあるとすれば、それを見つける必要があると思います。 (が、Javacriptのデバグの突っ込んでゆく技量が今ひとつなので、まだ追及していません) 今までこのサーバーで何度もPackやOOPS Xをインストールしましたが、この現象は初めてです。

再度確認したところ、chromeでこの現象が出ていますが、FirefoxやIE6だと正常ですね。。

サイトはこちらですので、chromeで実際の現象は見ていただけるかと思います。 http://www.naaon.com/xoopsx2/

(追伸) 他のPCからchromeで確認したら、現象が出ませんでした。 ローカル環境で何か起きているのかなー。 でも問題のPC+chromeで http://www.mc8.jp/mth/ を見ても問題出ないんですよね。

xoops123 commented 12 years ago

ご紹介いただいたテストサイトですが、WinPC ノートパソコンで確認しましたが、FireFoxでもChrome、IE8でも大丈夫でした。 パソコンの状態によって違うということなんでしょうか・・・?

xoops123 commented 12 years ago

ただ、theme.htmlを見ると、デフォルト状態で使っていないJSの記述が多すぎる気はします。 それが問題となっているとは思えませんが・・・xoopscube.netで使っている、multimenu?のドロップダウンメニュー用のJSとかがあって、ちょっと煩雑な感じでしょうか。テーマとした配布するなら、スライドショーの部分だけにするのが良いような気がしますね。

bluemooninc commented 12 years ago

デフォルトテーマを Boorstrap 対応にして JS を整理した方が良さそうですね。

http://blog.earthyworld.com/archives/bootstrap-from-twitter-jquery-ui/

これを入れて整理しようと思っています。あと右カラム強制なのも良くないですね。