日付のフォーマットの多言語対応 #4232
日付のフォーマットの多言語対応 #4232
Conversation
| chartData: Object, | ||
| date: { | ||
| type: String, | ||
| default: '' | ||
| }, |
There was a problem hiding this comment.
親コンポーネントからデータを渡されているのにも関わらず使用していなかったため追加。
| .text-nowrap { | ||
| white-space: nowrap; | ||
| } |
There was a problem hiding this comment.
表の中で変に日付が改行されしまうので、改行されないようにする共通のスタイルを追加。
| ja: dateTimeFormatsCommon, | ||
| en: dateTimeFormatsCommon, | ||
| 'zh-cn': dateTimeFormatsCommon, | ||
| 'zh-tw': dateTimeFormatsCommon, | ||
| ko: dateTimeFormatsCommon, | ||
| 'ja-basic': dateTimeFormatsCommon |
There was a problem hiding this comment.
いったん全ての言語で共通の設定とした。
utils/formatData.ts
Outdated
| export const getComplementedDate = (dateString: string): string => { | ||
| const dates = dateString.split('/') | ||
| if (dates.length !== 2) { | ||
| return dateString | ||
| } | ||
| const month = Number(dates[0]) | ||
| const date = Number(dates[1]) | ||
| const today = new Date() | ||
| const currentMonth = today.getMonth() + 1 | ||
| const currentDate = today.getDate() | ||
| let targetYear = today.getFullYear() | ||
|
|
||
| if (currentMonth <= month && currentDate <= date) { | ||
| targetYear = -1 | ||
| } | ||
|
|
||
| return `${targetYear}/${month}/${date}` | ||
| } |
There was a problem hiding this comment.
日付が3/10と言った形式だとDateオブジェクト取得する際に年が不明になってまい閏年などで日付がずれてしまう可能性があるため、以下のロジックで年を補完する処理を作成した。
- 年を含まないデータは1年以上データを保持しない。
- 1年以上保持しないのであれば、現在より古い日付のデータは去年のデータと推測できる。
- 最新のデータは翌日以降に取得するので、現在と同じ日付のデータも去年のものと考えられる。
- 一部だけ年を含むデータが入ってくることを考慮し、
/によりsplitした結果が�長さ２の配列にならない場合はそのままデータを返すようにした。
There was a problem hiding this comment.
ありがとうございます。
ファイル名なのですが formatDate.ts があるのでそちらに統合する形がいいかなと思います。
また、こちらの処理について、この関数を使うか元データの方で年部分まで保持する形にするか、一度検討してみたいと思います。
There was a problem hiding this comment.
@MaySoMusician formatDate.ts に統合しました！
|
外部の人間ですが一応指摘を... TimeBarChartを使っているグラフにおいてのみ、Y軸の目盛りがズレ、X軸の日付が消えます。英語の場合は月の部分も消えています。こちらを修正しなければならないと思います。 参考画像(画像はAndroid 9 / Google Chromeですが、Windows 10 / Google Chromeでも発生することを確認済みです。) |
|
また、都営地下鉄の利用者数の推移のグラフにカーソルを合わせても、ツールチップが表示されず、エラーを吐いていることを確認しています。 |
|
@y-chan ご確認いただきありがとうございます！ |
|
@MaySoMusician コンフリクトしていたようなので修正しておきました。 |
MaySoMusician
added
i18n
|
@MaySoMusician かしこまりました！ |
👏 解決する issue / Resolved Issues
📝 関連する issue / Related Issues
⛏ 変更内容 / Details of Changes
日付に関連するところを全て多言語対応しまいた。
📸 スクリーンショット / Screenshots
※全画面キャプチャすると多くなるので一部抜粋。