Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

ヒントボタンのタップ可能領域(Android) #985

Merged
4 commits merged into from
Jun 7, 2022

Conversation

unlimish
Copy link
Contributor

Issue 番号 / Issue ID

目的 / Purpose

タップ可能領域をアイコンの実サイズ以上に拡げることでアクセシビリティ/UXを向上させるため。

変更内容 / Changes

  • ? ボタンのPaddingを設定
    • Paddingを使用するために CachedButton から ImageButton へ置き換え

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[ x ] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[ ] Feature
[ x ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ x ] Other... Please describe: Design/UX Update

確認事項 / What to check

  • [ ]

その他 / Other information


Internal IDs:

CachedImage was replaced to ImageButton
@keiji keiji self-requested a review May 23, 2022 04:10
@keiji
Copy link
Collaborator

keiji commented May 23, 2022

@unlimish
Pull Requestありがとうございます。
デザインチームから「10pxでは範囲が広すぎて誤タップの原因となることから、5pxが妥当と考える」との意見が出ています。
ちなみに、ffimageloading:CachedImageをImageButtonに置き換えているのは何か理由がありますでしょうか。

また、コンフリクトの調整などをご対応頂けないでしょうか。
もしご負担が大きいようであれば、頂いているコミットを使いながら、一度featureブランチに取り込んでから、ぼくの方で調整することも可能です。

@unlimish
Copy link
Contributor Author

unlimish commented May 23, 2022

ボタンのpaddingサイズの件了解しました!

ffimageloading:CachedImageをImageButtonに置き換えていることに関しては、
ffimageloading:CachedImageにて、paddingが使用できなかったため、標準のImageButtonへ置き換えを行いました。

Confrictなどの調整に関しては解消してみようと思います!

@unlimish
Copy link
Contributor Author

unlimish commented May 23, 2022

@keiji

  • Padding 5pxへの修正
  • コンフリクト解消

完了しました!
お手隙の際にレビューお願いします。

@keiji
Copy link
Collaborator

keiji commented Jun 3, 2022

ありがとうございます。表示と動作を確認しました!
これで開発チームに引き継ぎます。

Android
Screen Shot 2022-06-03 at 17 30 57

iOS
Screen Shot 2022-06-03 at 17 34 59

Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

記録のためApproveを付けておきますね。実際の取り込みタイミングは開発チームにお願いして調整中です。

@ghost ghost merged commit 3826dad into cocoa-mhlw:develop Jun 7, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants