質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.35%

  • JavaScript

    11248questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML5

    2849questions

    HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

  • Chrome

    398questions

    Google Chromeは携帯、テレビ、デスクトップなどの様々なプラットフォームで利用できるウェブブラウザです。Googleが開発したもので、Blink (レンダリングエンジン) とアプリケーションフレームワークを使用しています。

素のJavascriptでWebComponentsを使う時、querySelectorが使えない

解決済

回答 2

投稿 2017/12/06 12:18 ・編集 2017/12/06 12:19

  • 評価
  • クリップ 1
  • VIEW 195

Fushihara

score 23

素のJavascriptでWebComponentsを使う時、templeteタグをquerySelectorする方法

github pagesでエラーが出るページを再現致しました。devtoolでどこでエラーが出ているかご確認いただけます。
https://fushihara.github.io/webcomponents-question-20171206/

HTMLElementを継承したXTestというクラスを定義し、クラスXTestをx-testという名前のカスタムタグとして登録しました。
XTestクラスの中で、カスタムタグの中身を作る時にtemplateタグを取得しcloneNodeしようと思ったのですが、document.currentScriptがnullになってしまいownerDocumentを取得する事が出来ませんでした。
XTextクラスの中でteplateタグを取る方法を教えて欲しいです。

サンプルコードの構造としては、重要なファイルは以下の3件。

/index.html
/web-component/custom-element.html
/web-component/custom-element.js

index.htmlでは以下の2行を書くだけでコンポーネントが使える。
インポート先のパス(この場合はweb-componentというディレクトリ名)が変わっても、cutstom-element.html jsファイルの挙動には影響が無いようにしたいので、index.htmlの中のscriptタグで何かをする という対応策は避けたいです。

<link rel="import" href="./web-component/custom-element.html">
<x-test id="a"></x-test>

custom-element.js もしくは custom-element.htmlだけの修正で、どこにもっていっても動くコンポーネントを作りたいです。

全ファイルの中身はgithubで公開しております。
インデックスとインデックスのjs、web componentのhtml とjsの4ファイルx2セットの差分です

https://github.com/fushihara/webcomponents-question-20171206

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • defghi1977

    2017/12/06 13:07

    この質問, 今後非常に価値が出る可能性があるため, 利用した技術(template要素, HTMLImports, ShadowDOM, CustomElement)についてのタグを追加したり, 本文・コードの補足などをしていただきたく.

    キャンセル

  • Fushihara

    2017/12/06 14:59

    はい、(今日中に)質問を編集して実際のコードを記入するように修正致します。

    キャンセル

回答 2

checkベストアンサー

+2

イベントハンドラやコールバック内でdocument.currentScriptを参照することは出来ません. なので, 一旦変数に取っておくようにします.

{
//ownerDocumentをこの場所で宣言すると問題ないが、ownerDocumentがグローバルにセットされてしまう
//→だったらブロックスコープで囲ってしまえば良い
const ownerDocument = document.currentScript.ownerDocument;
class XTest extends HTMLElement {
    constructor() {
        super();
        this.span = null;
    }

    /** DOMに要素が追加された際に発生するイベント */
    connectedCallback() {
        //コールバックからdocument.currentScriptにアクセスすることは出来ません
        //const ownerDocument = document.currentScript.ownerDocument;
        const template = ownerDocument.querySelector("#x-template");
        const instance = template.content.cloneNode(true);
        let shadowRoot = this.attachShadow({mode: 'open'});
        shadowRoot.appendChild(instance);

        this.span = shadowRoot.querySelector("span");
        shadowRoot.querySelector("button").addEventListener("click",()=>{
          this.span.innerText = this.span.innerText - 0 + 1;//innerTextは文字列なので一旦数値にしてから+1
        });
    }
    setMessage(msg){
      this.span.innerText = msg;
    }
}

customElements.define('x-test',XTest);
//外部に公開
window.XTest = XTest;
}

投稿 2017/12/06 12:31

編集 2017/12/06 14:47

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/12/06 14:37

    ありがとうございます。後出しで大変申し訳ありませんが、XTestクラスは外部から参照したい(実際はtypescriptで、index.jsの中から`document.querySelector("x-test") as XTest`としたい)ので、XTestクラスは現状のままグローバルにあって欲しいです

    キャンセル

  • 2017/12/06 14:49

    コードを更新しました. これでイケませんか?

    キャンセル

  • 2017/12/06 14:57

    ありがとうございます。
    自分の環境ではまだXTestクラス未定義エラーが出るのですが、おそらくtypescriptのコンパイラオプションやVSCodeの環境に起因する話になってしまいこの質問の範疇から逸脱してしまいましたので、
    本質問はdefghi1977さんの回答でクローズさせて頂きます。

    キャンセル

  • 2017/12/06 15:01

    typescriptというと型定義周りですかね(ワカッテナイ)

    キャンセル

  • 2017/12/06 15:06

    そうですね。window.XTest = XTest;と書いても、それは勝手にやってるだけなのでXTestがグローバルだと認識してはくれないみたいです。自分で型定義のファイルを書くか、それかtypescriptは別にエラーが出ても全部無視出来るので、おそらくエラー無効化したらちゃんと動くjsが出る気もします(それは試さない)。型が参照できないとかtypescriptでの都合でjsにコンパイル後は関係ないですし

    キャンセル

+1

 template 要素

templeteタグをquerySelectorする方法

問題の全容は分かりませんが、上記一文から推察できる内容としては下記事象でしょうか。

<template><x-test id="a"></x-test></template>
<script>
'use strict';
console.log(document.querySelector('#a'));  // null

var df = document.querySelector('template').content.cloneNode(true);
console.log(df);                     // #document^fragment
console.log(df.firstChild);          // <x-test id="a"></x-test>
console.log(df.querySelector('#a')); // <x-test id="a"></x-test>
</script>

template要素の内容はDOM上に現れませんので、document.querySelector で検索することは出来ません。
生成された DocumentFragment から querySelector を実行する事で期待通りに動作します。

 TreeWalker

別解として、TreeWalker を使う事でも、DocumentFragment からノードを検索する事が可能です。

var df = document.querySelector('template').content.cloneNode(true),
    tw = document.createTreeWalker(df, NodeFilter.SHOW_ELEMENT, function(node) {
  return node.id == 'a' ? NodeFilter.FILTER_ACCEPT : NodeFilter.FILTER_SKIP;
});
console.log(tw.nextNode()); // <x-test id="a"></x-test>

 document.currentScript

const ownerDocument = document.currentScript.ownerDocument;

このコードは何を意図しているのでしょうか。
下記コードは常に true なので、

console.log(document.currentScript.ownerDocument === document); // true

先の変数文は下記コードと同じ挙動になります。

const ownerDocument = document;

グローバル変数をローカル変数化する事でスコープチェーンを辿る回数が少なくなる為、document を何度も参照するならパフォーマンス向上が期待できますが、currentScript を経由する意味はないと考えます。
今一度、求めている動作を見つめなおしてみる事をお勧めします。

Re: Fushihara さん

投稿 2017/12/06 12:58

編集 2017/12/06 13:28

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/12/06 13:42

    実は「document.currentScript.ownerDocument === document」がtrueにならないケースがあるのです. それはHTMLImports仕様で外部HTML文書に含まれているscript要素をlink要素経由で実行しているケースで, 今回の質問は「HTMLImportsでインポートしているHTML文書に対応するdocumentオブジェクトを拾いたい」となります. WebComponentsを利用する場合, コンポーネントを一つのHTMLにまとめることが多く, このようなユースケースが発生します.

    キャンセル

  • 2017/12/06 14:44 編集

    今回の「document.currentScript.ownerDocument !== document」の件はまさにdefghji1977さんのご指摘の通りで、html importの中のscriptタグで読み込んでいる時はdocument.currentScript.ownerDocument.querySelector("template")でtemplateが取れるのに対し、XTestクラスのコンストラクタやインスタンスメソッドの中からtemplateを取る方法を知りたいです(XTestクラス以外にグローバル汚染を起こさず)

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

ただいまの回答率

91.35%

関連した質問

同じタグがついた質問を見る

  • JavaScript

    11248questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML5

    2849questions

    HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

  • Chrome

    398questions

    Google Chromeは携帯、テレビ、デスクトップなどの様々なプラットフォームで利用できるウェブブラウザです。Googleが開発したもので、Blink (レンダリングエンジン) とアプリケーションフレームワークを使用しています。