Stricter `datetime` validation
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 38/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- javascript
調査の方向性
まず datetime 属性のドキュメントと、issue に記載されている JavaScript Date コンストラクターの動作を確認し、その後、リンクされているクロスブラウザーの例を比較します。検証と regex-option の動作を定義し、一貫性のない日付文字列に対して何が起きるべきかも含めます。完了条件は、期待される動作が文書化され、影響を受けるすべてのブラウザーで検証されていることです。
索引モデルが issue の本文から書いたものです。
説明
The docs about the datetime attribute say:
This must be a valid ISO8601 DateTime
This statement is incorrect. Any value that can be passed to the Date constructor will work which presents a cross-browser issue because JS engines are inconsistent in which formats their Date constructor accept.
For example, the golang default string representation only parses in v8:
$ eshost -e "new Date('2009-11-10 23:00:00+00:00 UTC')"
#### JavaScriptCore
Invalid Date
#### spidermonkey
Invalid Date
#### v8
Wed Nov 11 2009 00:00:00 GMT+0100 (Central European Standard Time)
Also try this fiddle in multiple browsers.
How about adding an option attribute to pass in a validation regex into the element to validate the passed dates, when present? This would at least not make this issue missable by developers who only test in Chrome.
- 主要言語
- JavaScript
- スター
- 4k
- フォーク
- 194
- 平均マージ
- 4日 10時間
- マージ済み PR（30日）
- 5
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/relative-time-element のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
github/relative-time-element#383 · コメント 1 件 ·
-
Inconsistent use of "yesterday"再び着手できるかも @ayushman1210 が 331 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
github/relative-time-element#313 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
github/relative-time-element#307 ·
-
Astro Supportオープン
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
github/relative-time-element#306 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
github/relative-time-element#293 · コメント 2 件 · リアクション 5 件 ·
github/relative-time-element の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
521xueweihan/HelloGitHub#4025 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
yetone/magpie#2844 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
c1overzz/dsh-bili-scroll#22 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
maproulette/maproulette-frontend#2944 ·
メンテナーはふだん 1 日以内に返信