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

Moment.jsの利用方針の検討 #871

Closed
youchann opened this issue Jun 23, 2022 · 8 comments
Closed

Moment.jsの利用方針の検討 #871

youchann opened this issue Jun 23, 2022 · 8 comments
Assignees

Comments

@youchann
Copy link
Contributor

ref: #135
ref: #9

背景

具体的な日は不明瞭だが、2020年後半にMoment.jsがメンテナンスモードとなった。
https://momentjs.com/docs/#/-project-status/

We now generally consider Moment to be a legacy project in maintenance mode. It is not dead, but it is indeed done.

引用部周辺を見るとこのようなスタンスであることがわかる。

  • 新しい機能は追加しない
  • 既知で長年議論されている問題については対処しない
    • バンドルサイズが大きいこと
    • ロケールファイルの問題(要調査)
  • 一方でこの辺りは対処するよ
    • 重要なセキュリティ上の懸念があった場合
    • IANAタイムゾーンのデータ更新(タイムゾーンの規定みたいなものだろう)

この辺りの情報を加味しつつ、ingred-ui側はどう対応していくべきなのかを検討する。

@youchann youchann changed the title Momentやめるのか検討 Moment.jsの利用方針の検討 Jun 23, 2022
@maktak1995
Copy link
Contributor

公式が利用を推奨してない以上、そろそろ乗り換えはしておきたい。
候補として現実的なのは dayjsdate-fns

@maktak1995
Copy link
Contributor

またDatePickerに使用している react-dates は momentjs に依存しているので、こちらも選んだライブラリで構成されているものに置き換える必要がある

@maktak1995
Copy link
Contributor

比較するとこんなかんじ

date-fns

  • ネイティブのdate型が使える
  • スポンサーが多くコミュニティが活発

dayjs

  • interfaceが以前のと同じだから載せ替えが楽で軽い
  • 破壊的変更が最小限で済む
  • date-fnsよりも若い

@maktak1995
Copy link
Contributor

相談の結果、以下の観点で選定する

  1. 息の長そうなもの(また移行するのは面倒なので)
  2. 移行しやすい(インタフェースが似ている)
  3. 乗り換える予定のカレンダーライブラリとの親和性が高い

dayjsが若く、かつ移行しやすいのでこれを使うことにする

@maktak1995
Copy link
Contributor

https://grzracz.github.io/react-dayjs-picker/ これがいいかなー

@maktak1995
Copy link
Contributor

まともに使えそうなのが↑しか見つからないのでコレにする

@youchann
Copy link
Contributor Author

@maktak1995
このIssueだけみるとどうなったのかわからないので、いい感じに意思決定を書いてCloseしたりしておいてもらえると:pray:

@maktak1995
Copy link
Contributor

  • 後方互換性を考慮してdayjsへの置き換えを進めることに
  • momentjsに依存してるreact-datesも合わせて置き換える
  • いい代替がない & 同時にデザインも変えることになったので、DatePickerごと作り直す
  • 今後は DatePicker/DateRangePicker のリニューアル #975 で進めていく

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

No branches or pull requests

2 participants