Yuto Ashida

Results 15 comments of Yuto Ashida

なるほど、詳しくありがとうございます。 現状実装自体はうまくいっていて、後は最新developmentを適用してスタイルや表示速度の確認が必要という感じですね。 コンポーネントが増えすぎて対応が大変そうですが....(ちょうど #5064 でコンポーネントの統廃合に関して議論しているので、良ければ意見が欲しいです。落ち着いた段階でルビに関してマージできるとよいなと考えています。)

> また,Transifex の日本語辞書には > > - "自分や家族の症状に不安や心配があればまずは電話相談をどうぞ"(改行なし) > - "自分や家族の症状に不安や心配があれば\nまずは電話相談をどうぞ"(改行あり) > > の2つについて対応する文字列が指定されており,不自然である. これに関して、以前`auto-i18n`上で使われなくなったキーを削除しないようにしたことが原因ですが、確か理由が微妙なニュアンスの変更時などにも翻訳を引き継げるように、Transifexの文字列の一致率による翻訳の提案を使いたいから、のようだったと思います。ですがTransifexでは、1度翻訳された単語や文章は履歴として登録されるはずで、jsonのキーを消しても翻訳の提案は働くはずなので、未使用のキーの削除は復活させてもいいように思います。

論点としては同じコードが複数ある状態を許容して現在のまま行く(こちらはメンテナンスの面で問題がある)のか、propsや条件分岐を増やしてコードをまとめる(こちらは実行速度の差が出る、コードが複雑化する)のかだと思います。 メンテナーだけでなく、コントリビューターの皆様のご意見が聞きたい所存です。 現状まとめられるものは複数見受けられますが、すべてを探し切れていないので、後で挙げておきます。

今回の場合、主にグラフにおいての話題となります。そのほかのコンポーネントに関しても、統廃合できるのであればやるとよいのではないでしょうか(ただ、影響範囲がとてつもなく大きくなりそうな気はします....) グラフですと、formatDayBeforeRatioメソッドで言えば同じコードが各コンポーネントに点在していますし、あまりよろしくないと思います... displayInfoの差なんかは、propsで渡せるようにして、`component/cards`下のコンポーネントで作成して渡すようにしてもよいでしょう。 また、日別と累計の切り替えボタンを表示するか否かでもコンポーネントが増えているので、こちらもpropsで制御できるとよいのではないでしょうか。

> コンポーネントを組み合わせればグラフを実装できるようにしておくこと(柔軟性) この発想はなかったです...でもとても良いと思います。 計算ロジック等のVueファイル内でなくても処理できるものに関してはutils下のtsファイルにまとめることを考えています。それ以外はコンポーネントでまとめるのが良さそうです。 私自身まだまだVueを触り始めたばかりなので継承を使うのもありだということに気づきませんでしたが、これも良さそうですね。 私だけでは思い浮かばなかった統廃合案が出来そうです...! **Edited**: #5077 で日付計算関連がすでにまとめられていて、こんな感じを予想していました...!

では、少々長いですが統廃合案を挙げていきます。 日付の表記の違いなどありますが、 #5077 であらかた解決されそうなので含んでいません。 ## 廃止案 1. `MonitoringConfirmedCasesChart.vue` 1. 統合先: `MixedBarAndLineChart.vue` 2. 差分: 1. グラフの色 - propsで切り替えられるようにする。 2. デフォルトの数字フォーマッタ(小数点以下の処理) - そもそもpropsで渡せるようになっているので、都合のいいフォーマッタをデフォルトにする。 3. `tableLabels`propsの有無 - `required: false`にし、渡されている場合はそのまま使い、渡されていない場合は`dataLabels`を使うようにする。 4. データがnullの際の処理の有無 - nullの場合にグラフが描画されなくなる問題が起きることがあるので、`MixedBarAndLineChart.vue`に合わせてnullの際の処理を入れるようにする。...

@nard-tech 以前よりディレクトリの整理が必要という意見はお見かけしておりました...! 確かにそうですね、グラフの追加や廃止が現状収まっているので、そちらを優先した方がいいかもしれませんね。

@kaizumaki メンションありがとうございます。 かなり時間が経っているのは事実ですし、ディレクトリ構成の見直しでかなり改善されたというのは私も思っているので、改めて検討し直す方が良さそうですね。 そもそも統廃合が必要であるか、から考え直した方が良さそうです。 個人的にはディレクトリ構成の見直しによって、逆に統廃合が難しくなっているようにも感じるので、見送りと言うのも割と視野に入れるべきかなとは思っています。 少し考えて見ますね。

Hi, I'm Electrum-VIPS(Electrum for VIPSTARCOIN) developer.(VIPSTARCOIN is based on Qtum.) https://github.com/VIPSTARCOIN-electrum/electrum-vips Electrum-VIPS is based on Electrum (for Bitcoin) Version 3.3.8 and Qtum-Electrum Version 0.18.14. This state can support lightning. I...

Wow! That's nice idea! I'm waiting for your PR!