os0x / autopatchwork Goto Github PK

https://github.com/os0x/AutoPatchWork/blob/master/AutoPatchWork.safariextension/includes/AutoPatchWork.js#L702
全体の高さが変わらないとき、ここ、702行目の判定が真になって、3ページ目以降が存在しても正常終了します。
サイトバーがpageElementより倍以上の高さがあるようなページで発生します。

以下のSITEINFOがAutoPatchWorkでは動作しません。
http://wedata.net/items/60489

なお、Firefox、ChromeのAutoPagerizeでは動作したので、AutoPatchWorkでの問題です。

おそらく、NAVERまとめの記事での対策が誤爆しているのではないでしょうか。

こんにちは。
サイト固有かつFirefox+AutoPagerizeなどでも再現する問題なので、AutoPatchWorkのIssueとするのが適切かどうかわかりませんが、SITEINFOの修正だけでは解決できそうにない問題のような気がしたのでここに書き込ませていただきました。
もしSITEINFOの修正だけで済む問題だったらすみません。

Amazon.co.jpではある条件下 (詳細は不明ですが、手元ではAmazonにログインすると再現しました) でページャが新しいタイプのものに置き換えられるらしく、今回の問題はこの新しいページャで起きる問題です。
新しいページャでは、hash fragmentを利用して動的にページ切り替えを行うようになっており、それに伴ってnextLinkで指定されているpagnNextLinkのhrefが#から始まるものに変更されています。そのため、AutoPatchWorkが動作するとそのとき開いていたURLの内容がそのまま継ぎ足されてしまいます。SITEINFOを修正できないかと思ったのですが、DOMツリー上にnextLinkとして使えるような代わりの要素が見当たらなかったので、対処方法がわかりませんでした。
なお、問題が起きるのは最初の1回だけで、2回目以降は正しいページが継ぎ足されます。これは、pagnNextLinkのhrefはHTMLの時点では#から始まるものにはなっておらず、JavaScriptで動的に変更しているためだと思われます。

よろしくお願いします。

On Safari (7.0.1 currently) AutoPatchWork seems to have problems respecting the settings from time to time. I mean the checkboxes and radios remain checked/unchecked correctly, but AutoPatchWork often ignores them, causing unwanted behaviors, and requires user to manually toggle them again just to make it work.

Any ideas?

Hi there,

the extension does not work on https://encrypted.google.com . It only works on https://google.com search. Is there any possibility of supporting the encrypted version? There are differences between the two even though both are HTTPS channel.

http://wedata.net/items/27628?rev=121630
既にロールバックしていますが、この状態のSITEINFOのときに
Scrollの含まれるほうのexampleUrlにアクセスすると、
2ページ目以降で、1ページ目と同じ画像がロードされます。

AutoPagerizeにおけるresolvePathに相当する処理が必要と思われます。

The calc_remain_height shouldn't accept nodes that don't have visual representation, like script or style, but it does and it breaks extension on some pages. Here is the corrected version:

        function calc_remain_height() {
            var rect = null, bottom = null, _point = insert_point;
            while (_point) {
                if (typeof _point.getBoundingClientRect === 'function') {
                    rect = _point.getBoundingClientRect();
                    if (rect && !(rect.top === 0 && rect.right === 0 && rect.bottom === 0 && rect.left === 0)) break;
                    else rect = null;
                }
                if (_point.nextSibling) _point = _point.nextSibling;
                else break; 
            }
            if (rect) {
                bottom = rect.top + window.pageYOffset;
            } else if (append_point && typeof append_point.getBoundingClientRect === 'function') {
                rect = append_point.getBoundingClientRect();
                bottom = rect.top + rect.height + window.pageYOffset;
            }
            if (!bottom) bottom = Math.round(rootNode.scrollHeight * 0.8);
            return rootNode.scrollHeight - bottom + options.BASE_REMAIN_HEIGHT;
        }

• https://chrome.google.com/webstore/detail/aeolcjbaammbkgaiagooljfdepnjmkfd

Just I realized that it's 404. What's happened?

Just a heads-up: Google automatically unchecks (disables) the extension now. I was wondering why it wasn't working.