-
Notifications
You must be signed in to change notification settings - Fork 112
設定ページに項目「開発に参加する」を作成 #154
base: develop
Are you sure you want to change the base?
設定ページに項目「開発に参加する」を作成 #154
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Descriptionからリンクしていただいている #84 については、COCOA自体のライセンス表示も含まれたもので、このPull RequestではCloseできないので キーワード Close だけ外しておいてください。
ご指摘ありがとうございます。外しておきました。 |
言い回しですが,このような表現はどうでしょうか。#154 (comment) と文意はおおむね一致するはずです。 日本語
English
|
[nit] デザイン面で提案です。 |
以下の様に書き換えるのはどうでしょうか? 日本語
English
変更点
GitHub の案内書きについて「開発に参加する」の文言が重複していたので、「訪れてみてください」に変更しました。 まだコミットは作成していません。 |
アプリ内に URL 文字列が登場していないか探してみましたが,「お問い合わせ」ページにありますね。以下のスクリーンショットからわかるように,こちらは URL 部分に下線が引いてある状態です。装飾の統一が可能ならば,その方がよろしいかと存じます。 (当該ページと設定ページとでは実装方法が異なるため,あくまで参考程度の情報ですが……) cocoa/Covid19Radar/Covid19Radar/Views/HelpPage/InqueryPage.xaml Lines 67 to 74 in 872cc86
|
リンクの装飾については対応しました。 |
[提案] 変更後のスクリーンショットも共有していただけるとレビュワー含め確認しやすいかも? |
[提案] 文字列リソースの追加は別PRに切り出してみませんか? 後の人がこのPRを見た時に理解しやすいかも。 |
@fumiya-kume 文字列リソースの部分だけを別のPRに分割するとマージ作業が増えますし、情報が分散してしまうのでこのままにしておきます。 |
ありがとうございます。 一方、既存画面のレイアウト変更+文言変更のようなパターンだとそれぞれをPull Requestに分けてもらった方が話が進みやすいかもしれませんね。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
本当にお手数をかけてしまいますが、MATの廃止に伴うコンフリクトの解消、なにとぞお願いします!
ライセンスは重要なのでリンクより上に持ってきた方がいいと思います。 また配布版に記載するテキストについては、FireFoxとかの文章を参考に、「COCOAは Mozilla Public License の条項下で提供されます。ソースコードは次のリンクよりダウンロードできます」でどうでしょうか(いっそアタリの文章にしておいて開発チームのレビューに委ねる方がいいかもしれません)。 |
ライセンスの場所は下記の画像の赤線の部分が良いでしょうか?
提案された文章と #154 (comment) の文章と繋げて下記の様にしてみました。前述の文章を赤線の部分に、後述の文章を青線の部分に配置しています。
|
MATに関するコンフリクトを解消しました。差分が見易くなりましたね。 |
Issue 番号 / Issue ID
目的 / Purpose
破壊的変更をもたらしますか / Does this introduce a breaking change?
Pull Request の種類 / Pull Request type
検証方法 / How to test
コードの入手 / Get the code
コードの検証 / Test the code
確認事項 / What to check
その他 / Other information