Skip to content

Rename String.prototype.contains to includes - #210

Merged
syoichi merged 1 commit into
tombfix:masterfrom
wantora:fix_contains
Jul 24, 2016
Merged

Rename String.prototype.contains to includes#210
syoichi merged 1 commit into
tombfix:masterfrom
wantora:fix_contains

Conversation

@wantora

@wantora wantora commented May 13, 2016

Copy link
Copy Markdown

Firefox 48.0a2 でポストフォームを表示した際に String.prototype.contains が存在しないためエラーが発生しています。

String.prototype.contains は Firefox 40 で includes に改名され、 Firefox 48 で contains が削除されたようです。

@syoichi

syoichi commented May 16, 2016

Copy link
Copy Markdown
Member

Pull Requestありがとうございます!
対応が遅れていてすみません。可能であれば近日中に対応したいと思います。

@syoichi syoichi added the bug label Jul 24, 2016
@syoichi
syoichi merged commit 002f31c into tombfix:master Jul 24, 2016
@syoichi

syoichi commented Jul 24, 2016

Copy link
Copy Markdown
Member

遅くなりましたが、動作を確認しました。ありがとうございました!

@wantora

wantora commented Jul 24, 2016

Copy link
Copy Markdown
Author

対応ありがとうございます!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants