notifications.php JavaScript @ Safari

• Auswertung: jidaikobo
• Verantwortlicher: tom_g3x
• Typ: XOOPSCube2.1 Bugs
• Status: Geschlossen
• Komponente: (Keine)
• Meilenstein: (Keine)
• Priorität: 7
• Schweregrad: 5 - Mittel
• Lösung: Gefixt
• Datei: Keine

Details Antworten

Safariで、notifications.php をみているとき、個別に
チェックする分にはおかしくないんですが、モジュールごと
のグループをまとめてチェックするとき、なんだかナダラカ
に各チェックボックスのオンオフを繰り返してしまい、期待
通りに動作しません。
報告だけですみませんが、とりいそぎ、

Ticket-Verlauf (3/18 Historien)

2007-03-20 12:16 Aktualisiert von: minahito

• Summary Updated

Kommentar Antworten

Logged In: YES
user_id=8353

[EN][REPORT TRANSLATION]
When I view notifications.php with Safari, I can not check
module groups. It seems that Javascript's error.
(But, I can check each box)

2007-03-20 12:17 Aktualisiert von: minahito

• Priorität Update from 5 - Mittel to 7
• (Kategorie change on Tracker)

2007-03-20 12:33 Aktualisiert von: minahito

• Verantwortlicher Update from (Keine) to tom_g3x
• Lösung Update from Keine to Accepted

Kommentar Antworten

Logged In: YES
user_id=8353

Tom, please check this. (Do you have Safari?)
See #10118.

[JA]
Tom さんお願いします。(Safari持ってましたよね？^^;）
#10118 が関係しているかもしれません。

2007-03-20 12:52 Aktualisiert von: tom_g3x

Kommentar Antworten

Logged In: YES
user_id=6745

Sorry....... I don't have Apple PC.

2007-03-20 15:23 Aktualisiert von: minahito

• Verantwortlicher Update from tom_g3x to (Keine)

Kommentar Antworten

Logged In: YES
user_id=8353

Uu...

2007-03-20 17:04 Aktualisiert von: minahito

Kommentar Antworten

Logged In: YES
user_id=8353

We may not be able to check this & fix this, because there
is not a developer who can have Mac and can understand
Javascript. If you can fix this, please contribute your patch!

[JA]
この問題を再現し、修正することが出来ないかもしれません...と
いうのも Mac を持っていて、かつ、Javascriptができるという条
件を満たす開発者がおらんのです。どなたかこれを修正可能であれ
ば、パッチをくださいませ。 m(__)m

2007-03-20 19:00 Aktualisiert von: jidaikobo

Kommentar Antworten

Logged In: YES
user_id=24768

I'm not good at JavaScript :-(
so, sorry.
someday i'll try to fix again.

[JA]
すんません、JavaScript よくわかんないので、いまのところ措い
ておきましょう。
でも、当てずっぽうですみませんが、#10118 の id をユニークに
するというのをしたら、なおったり……しないか……。
僕、意識しておきますので、また改めてみてみます。

2007-03-30 18:34 Aktualisiert von: tom_g3x

• Verantwortlicher Update from (Keine) to tom_g3x

Kommentar Antworten

Logged In: YES
user_id=6745

[ #10118 ] で、ユニークなIDを付けてみました。
もしかしたら、それが原因だったのかもしれません。
Snapshot などで一度確認していただけないでしょうか？

[en]
Unique ID was put up by [# 10118].
It might have been a cause possibly.
Could you check it with Snapshot etc. once?
[/en]

2007-03-30 18:43 Aktualisiert von: jidaikobo

Kommentar Antworten

Logged In: YES
user_id=24768

I have my sister's marriage this weekend. So, I'll check
sometime later, sorry!

[JA]
対応ありがとうございます！　でも、すんません、今週末に妹の結
婚式があって、ちょっとすぐにチェックできなさそうなんです。し
ばらくお待ちくださいませ～。

2007-04-04 09:00 Aktualisiert von: jidaikobo

Kommentar Antworten

Logged In: YES
user_id=24768

I checked snapshot_2007404-0812. But Javascript didn't work
well at Safari.
One thing I have to add the information.
notification.php's javascript works in the part. the button
to check all checkbox works well. trouble occurs just in
each modules' button.

[JA]
snapshot_2007404-0812 で Safari の Javascript 動作をチェック
してみました。
しかし，notification.php のモジュール単位でのまとめてチェッ
クは機能しませんでした。しかし，全体にチェックをかけるボタン
はうまくいきます。
引き続き気にして（あわよくば直せればいいのですが）いきます
が，チェックができなくなる訳ではないので，致命的なバグではな
いように思っています……。

2007-04-05 00:28 Aktualisiert von: tom_g3x

Kommentar Antworten

Logged In: YES
user_id=6745

なんどもすみません、もう一度修正かけてみました。
お時間ありましたら、再度チェックしていただけないでしょうか？

[en]
comitted teat patch again.
plz check it.
[/en]

2007-04-05 08:40 Aktualisiert von: jidaikobo

Kommentar Antworten

Logged In: YES
user_id=24768

I tried, but didn't work well.
see movie plz(120KB).
http://www.jidaikobo.com/xoops/FinderScreenSnapz002.mov

[JA]
ためしてみました～。snapshot_2007405-0412 です。
でも、ざんねんながらうまくゆきませんでした。
スクリーンキャプチャを取ってみたので、よかったらごらんください。

2007-04-05 17:58 Aktualisiert von: okuhiki

Kommentar Antworten

Logged In: YES
user_id=13441

remove to Line 141.
What operation do it become?

[code]e.click(); e.click(); // ...[/code]

[ja]141 行を削除するとどのように成りますか。[/ja]

2007-04-06 06:59 Aktualisiert von: jidaikobo

Kommentar Antworten

Logged In: YES
user_id=24768

thank you for advising, okuhiki-san.
I searched the line from entire files, but could not find it.
which file should i edit?

[JA]
okuhikiさん、アドバイスありがとうございます。
が、最初 notification のテンプレートか、php 本体の 141 行目
をいじるんだろうなと探してみたんですが、みつからず、全ファイ
ル内を検索かけたんですが、「e.click」のみでもなにも引っかか
りませんでした。
お恥ずかしいんですが、目的ファイルを教えていただけますか？
いじったのは、snapshot_2007406-0412 でした。

2007-04-06 07:51 Aktualisiert von: okuhiki

Kommentar Antworten

Logged In: YES
user_id=13441

sorry.
/include/xoops.js Line 141.

2007-04-06 07:59 Aktualisiert von: jidaikobo

Kommentar Antworten

Logged In: YES
user_id=24768

thanks and worked well at safari!

[JA]
あ、ファイルの検索対象を .php と .html だけにしていました。
お恥ずかしい……。
当該箇所をハツったら Safari でうまくいきました！ Camino でも
ばっちり動いています。

2007-04-06 18:28 Aktualisiert von: okuhiki

Kommentar Antworten

Logged In: YES
user_id=13441

動いて何よりです。
Windows の ie firefox 共に、無くても(有っても)稼動します。

この(141)行が実行しているのは、
該当チェックボックス上にマウスカーソルを置いたと
仮定しマウスの左ボタンを 2 回クリックです。

怪しく推測するに、昨今のマシンの処理能力では、
適度な間隔を取らずに 2 回のクリックを実行すれば、
ダブルクリック、1 回のクリック、などと判断されて
いるかも知れませんね。

スクリプトの書かれた経緯が不明なので対処は、
対処その 1
141 行の削除。

対処その 2
既存関数は以前の状態に戻し、141 行部分を除いて
新規関数(xoopsCheckGroupAll などが判り易いかも)
を作成する。

対処その 3
クリックの間に適当な間隔を取る。

可能であれば、スクリプトの作成経緯などをご存じの方に
問い合わせてから判断頂く事が良いと思われます。

2007-04-07 21:22 Aktualisiert von: tom_g3x

• Ticket Close date is changed to 2007-04-07 21:22
• Lösung Update from Accepted to Gefixt
• Status Update from Offen to Geschlossen

Kommentar Antworten

Logged In: YES
user_id=6745

okuhikiさん、ありがとうございます。
では、line141 をコメントの形で消して、
一旦、close って事にしておきます。

[en]
thank you okuhiki.
Line 141 is deleted and committed.
[/en]