dara-j

このブログのサイドバーに「最近のブックマーク」というのを貼っている。自分のtumblrのlinkのみをjson apiで抽出して並べる、というものだ。

で、だいたいtumblrにlinkをpostするときは、何についてのページなのかを忘れないように引用をつけたりしていて、「最近のブックマーク」ではその部分の表示をトグル切り替えするようにしている。

さて、本題。

なんとなくのノリで、「最近のブックマーク」をちょっと調整していたのだが、Safari for Winではトグル切り替えがまったく動作しない。

「最近のブックマーク」の独立版であるこちらのページは問題なくトグル切り替えができているし、他のブラウザでも特に問題がない。

トグル切り替え部分のイベント登録には、prototype.jsのEvent.observe()を使っているのだが、そこに登録しているイベントハンドラで、UAがSafariだった場合にalert()するデバッグコードを仕込んだところ、これも実行されない。どうやらEvent.observeでのハンドラ登録がうまくいっていないようだ。

手持ちのSafari for WinはDebug メニューを有効にしてあるので、JavaScript Consoleを表示したところ、

TypeError: Value undefined (result of expression this.attachEvent) is not object.
http://s.asablo.jp/js/expandimage.js  Line: 49

この「expandimage.js」、アサブロの画像表示周りのスクリプトらしい（自分ではあまりアサブロに画像登録をしていない。外部に置いてそっち参照させるので）のだが、この49行目前後を見てみると、

48: HTMLElement.prototype.addEventListener = function(name,func,dir) {
49:  this.attachEvent(name, func);
50: }
 

if (! (document.body && document.body.insertAdjacentHTML) || asablo.noHTMLElement) {

• HTMLElementオブジェクトが存在しない場合は定義する
• HTMLElement.prototype.insertAdjacentHTMLの定義
• HTMLElement.prototype.addEventListenerの定義

...ちょっとひどくないか？

上記の処理の対象となるのは

• document.body.insertAdjacentHTMLがないブラウザ（＝Firefox、Safari）
• asablo.noHTMLElementがtrueなブラウザ（＝IE）

幸いというか、IEでのinsertAdjacentHTMLの定義はDOMとは無関係なダミーのHTMLElementに対して行われるため問題が起きず、FirefoxでのaddEventListenerの定義は他のDOM要素に（なぜか）影響を与えていないのだが、SafariはがっつりとaddEventListenerの上書きの影響をこうむっちゃっている。

こういう処理はもうちょっと丁寧に状況分けしないとまずいでしょ。