AppLinks セクションを追加 #213
AppLinks セクションを追加 #213
Conversation
|
Visit the preview URL for this PR (updated for commit 3838355): https://flutterkaigi-2023-preview--pr213-feature-app-links-h26chbxp.web.app (expires Sat, 28 Oct 2023 08:19:31 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: ec86c184da0140e1a174c92c6d39f7a77720a028 |
There was a problem hiding this comment.
↓でダウンロードしたものを Figma に配置して、上下のパディングを削除してアプリで使いやすいようにしています。
https://play.google.com/intl/en_us/badges/?hl=ja
There was a problem hiding this comment.
↓でダウンロードしたものを Figma に配置して Google Play と同じ画像形式で扱えるように png としてダウンロードしてアプリで使いやすいようにしています。
https://developer.apple.com/jp/app-store/marketing/guidelines/
| @@ -19,7 +19,7 @@ class _Accent { | |||
| ], | |||
| ); | |||
|
|
|||
| Gradient get secoundary => const LinearGradient( | |||
| Gradient get secondary => const LinearGradient( | |||
There was a problem hiding this comment.
Typo があったため、ついでに修正しました。
There was a problem hiding this comment.
Typoの修正いただきありがとうございます！！
全然気づけていませんでした🙇♀️
| Gradient get diamond => const LinearGradient( | ||
| colors: [ | ||
| Color.fromARGB(255, 155, 202, 240), | ||
| Color.fromARGB(255, 225, 253, 255), | ||
| Color.fromARGB(255, 155, 202, 240), | ||
| ], | ||
| ); | ||
|
|
There was a problem hiding this comment.
未使用で今後も使用する予定がないため、ついでに削除しました。
| ScreenSizeType.large || ScreenSizeType.medium => 40.0, | ||
| ScreenSizeType.small => 24.0, | ||
| }; | ||
| return Wrap( |
There was a problem hiding this comment.
より小さい端末では、自動的にリンクを縦に並べられるように Wrap を使用しました。
| final imageButton = Material( | ||
| color: Colors.transparent, | ||
| child: Ink.image( | ||
| image: AssetImage(assetName), | ||
| fit: BoxFit.fitHeight, | ||
| child: InkWell( | ||
| onTap: onTap, | ||
| borderRadius: BorderRadius.circular(8), | ||
| ), | ||
| ), | ||
| ); |
There was a problem hiding this comment.
公式ドキュメントに記載されている通り、画像にエフェクトを被せられるようにこのようにしています。
ただし、角丸がいい感じに自動的につけることができなかったため、違和感なさそうな固定値を設定しています。
FYI:
| final aspectRatio = switch (_type) { | ||
| _AppLinkType.appStore => 148 / 54, | ||
| _AppLinkType.googlePlay => 182 / 54, | ||
| }; |
There was a problem hiding this comment.
Figma から落としてきた画像のサイズで縦横比を計算しています。
| ScreenSizeType.large || | ||
| ScreenSizeType.medium => | ||
| 'FlutterKaigi 2023 のカンファレンスアプリをダウンロード', | ||
| ScreenSizeType.small => 'FlutterKaigi 2023 の\nカンファレンスアプリをダウンロード', |
There was a problem hiding this comment.
小さい端末では意図的に「FlutterKaigi 2023 の」の後に改行を入れて、ヘッダーやボタンのサイズ感に合わせるようにしています。
There was a problem hiding this comment.
とてもいい感じに実装されていますね！ありがとうございます！！！
| @@ -19,7 +19,7 @@ class _Accent { | |||
| ], | |||
| ); | |||
|
|
|||
| Gradient get secoundary => const LinearGradient( | |||
| Gradient get secondary => const LinearGradient( | |||
There was a problem hiding this comment.
Typoの修正いただきありがとうございます！！
全然気づけていませんでした🙇♀️
| ScreenSizeType.large || | ||
| ScreenSizeType.medium => | ||
| 'FlutterKaigi 2023 のカンファレンスアプリをダウンロード', | ||
| ScreenSizeType.small => 'FlutterKaigi 2023 の\nカンファレンスアプリをダウンロード', |
There was a problem hiding this comment.
とてもいい感じに実装されていますね！ありがとうございます！！！
|
いま確認して無事にストアに遷移できたのでマージしてしまいます！ |
Issue
https://github.com/FlutterKaigi/TasksBoard/issues/169
Overview (Required)
AppLinks セクションを追加しました。
iOS アプリのリンクはアプリチームから共有いただいたものなのでおそらく問題ないかと思いますが、公開され次第確認してマージ予定です。
Screenshot