Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

indicator、viewのターゲット指定ではイベントハンドラ記述と同様のセレクタを使用できるようにする #349

Closed
fukudayasuo opened this issue Jun 12, 2014 · 1 comment
Assignees
Milestone

Comments

@fukudayasuo
Copy link

今の仕様だと、コントローラのインジケータのtarget、view.apend/update/prependの第1引数は共に、グローバルセレクタ記述で記述することができるが、"window."及び"navigator."で始まるグローバルセレクタはエラーになる。
(コントローラのindicator,viewでない、h5.ui.indicatorやh5.core.viewはそもそもグローバルセレクタを解釈しない)

h5.core.controller('body', {
  __name:'a',
  __ready: function(){
    this.view.register('tmpid', 'hoge');
    this.view.append('{window.document.body}', 'tmpid');
  }
});
// [ERROR]11:13:15,735: コントローラ"a"の__ready内でエラーが発生したため、コントローラの初期化を中断しdisposeしました。
// Uncaught Error: update/append/prepend() の第1引数に"window", "navigator", または"window.", "navigator."で始まるセレクタは指定できません。(code=6000)

インジケータのtargetについても同様のチェックをしており、同じエラーが投げられる。

コントローラのindicatorもviewも、ターゲットの指定はコントローラのイベントハンドラ記述と同様に、"window."または"navigator."で始まるグローバルセレクタを解釈できるようにする。

fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Jun 12, 2014
…うにしました。

(getTarget()を修正し、第3引数(isTemplate)を削除、第2引数を探索の起点にするコントローラにしました。)
@fukudayasuo fukudayasuo self-assigned this Jun 12, 2014
fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Jun 12, 2014
テストケース『view.append()に指定されたDOM要素が{window*},{document*}である時にエラーが発生すること』を削除
fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Jun 12, 2014
@simdy simdy added this to the v1.1.12 milestone Jun 16, 2014
fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Jun 17, 2014
コントローラのviewとindicatorについて、ターゲットの指定がコントローラのイベントハンドラ記述のルールと同じであることを確認するためのケースを追加しました。
@simdy simdy closed this as completed Jun 17, 2014
@fukudayasuo
Copy link
Author

テストケースに不備があったため修正します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants