URL末尾のハッシュが表示領域に入った要素のIDに変更されるようにしたい

Question

###前提・実現したいこと
要素（entry）が画面内に入るたびに
test.html#entry1、test.html#entry2、test.html#entry3...と
URL末尾のハッシュが各要素のIDに変更されるようにしたいと思っています。

###該当のソースコード

<div class="entry" id="entry1">
	<p>記事1記事1記事1記事1</p>
</div>

<div class="entry" id="entry2">
	<p>記事2記事2記事2記事2</p>
</div>

<div class="entry" id="entry3">
	<p>記事3記事3記事3記事3</p>
</div>

###試したこと
inview.jsを用いて下記のようにしましたがうまくいきませんでした

$(window).scroll(function (){
$('.entry').on('inview', function() {
window.history.pushState(null, null, "XXX.com/works/ + idName");
});
});

いい方法があれば教えていただけますでしょうか。

###補足情報(言語/FW/ツール等のバージョンなど)
http://gimmicklog.main.jp/jquery/581/

Comments

[kei344]

質問文のコードはそれぞれコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

[facetamamu]

すみません、修正いたしました。ありがとうございます。

Answer 1

IntersectionObserver

まだ、不安定な仕様ですが、IntersectionObserverによって要素が画面内に入ってきたタイミングを捕捉できるようです。
Polyfill もあるようです。

• 画面内に要素が入ってきた事を検知 (IntersectionObserver) - JSFiddle
• IntersectionObserver - Web API インターフェイス | MDN
• IntersectionObserver - Can I use...
• IntersectionObserver/polyfill at gh-pages · w3c/IntersectionObserver

<script>
'use strict';
(function () {
  function createElements (doc, length, tagName, idPrefix, observer) {
    var df = doc.createDocumentFragment(),
        baseElement = doc.createElement(tagName),
        element, i = 0;

    while (i < length) {
      element = baseElement.cloneNode(true);
      element.id = idPrefix + ++i;
      element.textContent =  '#' + idPrefix + i;
      observer.observe(element);
      df.appendChild(element);
    }

    return df;
  }

  function main () {
    var doc = this.document, df;

    const observer = new IntersectionObserver((entries) => {
      for(const entry of entries) {
        if (entry.isIntersecting) {
          console.log(entry.target);
        }
      }
    });

    df = createElements(doc, 1000, 'p', 'foo', observer);
    doc.body.appendChild(df);

  }

  main.call(this);
}.call(this));
</script>

jquery.inview.js

jquery.inview.js では、scroll, resize, scrollstop イベントによって画面の変化したタイミングで要素位置を再計算して捕捉しているようです。

• 画面内に要素が入ってきた事を検知 (jquery.inview.js) - JSFiddle
• protonet/jquery.inview: A jQuery plugin that adds a bindable 'inview' event for detecting when an element is scrolled into view.

<script src="../lib/other-work-lib/jquery-3.1.1.js"></script>
<script src="./jquery.inview-master/jquery.inview.js"></script>
<script>
'use strict';
(function () {
  function createElements (doc, length, tagName, idPrefix) {
    var df = doc.createDocumentFragment(),
        baseElement = doc.createElement(tagName),
        element, i = 0;

    while (i < length) {
      element = baseElement.cloneNode(true);
      element.id = idPrefix + ++i;
      element.textContent =  '#' + idPrefix + i;
      df.appendChild(element);
    }

    return df;
  }

  function handleInview (event) {
    console.log(event.target);
  }

  function main () {
    var doc = this.document, df = createElements(doc, 1000, 'p', 'foo');

    jQuery(df.children).on('inview', handleInview);
    doc.body.appendChild(df);

  }

  main.call(this);
}.call(this));
</script>

history.pushState

history.pushState はJavaScriptで動的にページを書き換えたタイミングでURLの履歴を残す目的として、仕様化されました。
副次的効果としてロケーションバー(アドレスバー)の値が書き換えられますが、履歴に残すのが本来の目的です。
質問文にある要件を history.pushState で実装した場合、ページの上スクロール/下スクロールを繰り返すだけで多数の履歴が残り、[戻る], [進む] の履歴も合わせて増えます。

• ユーザがページ遷移前のページに戻す為に、[戻る] ステップ数が多くなる
• 多数の履歴が残り、履歴から検索をするのが困難になる

ユーザビリティが低下する実装となりますので、要件を見直す事をお勧めします。

Re: facetamamu さん

Answer 2

URLを書き換えるだけならこう変更（自動スクロール修正版）

var flag = false;
(window).scroll(function (){ flag = true: );
$(function (){
    $('.entry').on('inview', function() {
        if (flag) { location.hash= "HASH-NAME"; }
        flag=false;
    });
});

以上

Comments

[退会済みユーザー]

注意）　イベントキーは質問者のが正解しているという前提

[kei344]

scrollイベントの中でonを使うと大量にイベント登録してしまいますよ。

[退会済みユーザー]

うん。
イベントキー（発火）は質問者のコードのまま

[think49]

> location.hash= "HASH-NAME"
location.hash は強制的にスクロールさせます。
スクロールする度にページの #hash がある場所に強制スクロールされるので、正常にスクロールできなくなるのでは…。

[退会済みユーザー]

hash が同じ値だと hashchange が再発火しないはず

[退会済みユーザー]

とりま、オリジナルからイベントの登録を修正

[facetamamu]

ご回答ありがとうございます。希望の動作にならずでした。おっしゃいますように、そもそものイベントの書き方が試行錯誤のものだったのでここからダメなようです...。inviewは必須ではないのですが、何か他にいい方法があるでしょうか

[think49]

To: asahina1979 さん

> hash が同じ値だと hashchange が再発火しないはず
逆に言えば、hash が変わった場合は強制スクロールしてしまいませんか？

■scrollイベントをフックした場合(編集前のパターン)
1. #hash-a が画面内にある
2. 下スクロールされ、#hash-a が画面外になり、#hash-b が画面内にある
3. location.hash= "#hash-b" によって、#hash-b の位置に強制スクロールされる
4. 下スクロールされるも、#hash-b がまだ画面内にある為、3. に戻る

編集後のパターンではスクロール禁止にはなりませんが、描画される要素が切り替わったタイミングで強制スクロールされる問題は残っています。

[think49]

> 編集後のパターンではスクロール禁止にはなりませんが
試してみたところ、そうでもありませんでした。
画面内に複数の要素が含まれるパターンが考慮されていないようです。
https://jsfiddle.net/rwmu0wan/3/

[退会済みユーザー]

自動スクロールだねぇ・・・