電卓ダイアログを小さい画面に収め、会計中にぐるぐるを出す - #52
Merged
Merged
Conversation
- ごうけい・おあずかり・おつり/たりない を大きな文字で常時表示し、数字を押しながら比較できるようにする - 預かりが足りないときは たりない と不足額をエラー色で、足りたときは おつり を強調色で表示する - 会計結果はおつりを主役にした専用ダイアログにし、ごうけい・おあずかりを補足に降格する - タブレットの広い画面では金額と数字パッドを左右2カラムに並べる - 文字崩れを防ぐため、実測して枠に収まる最大のタイポグラフィを選ぶ - バージョンを 1.0.10 に上げる
Robolectric のデフォルト端末は 320x470dp と狭く、電卓ダイアログの数字キーが 縦スクロールの外に置かれてタップが届かないため、預かり金額が加算されず 金額表示の検証がすべて失敗していた。実際の利用端末に合わせて MediumTablet の画面サイズで実行し、キーが表示されていることも明示的に確認する。
- もどる・かいけいのフッターをスクロール領域の外に固定し、常に押せるようにする - 残りの高さからキーパッドの大きさを決め、Nexus 7 の横向きでも画面に収まるようにする - 金額の文字サイズを横幅だけでなく高さでも判定する - おつり画面のかいけい押下からレシート発行が終わるまで進捗インジケータを表示し、二重送信を防ぐ
Contributor
UI差分main_screen_calculator
main_screen_calculator_shortage
全画面スクリーンショットmain_screen
main_screen_account_result
main_screen_calculator
main_screen_calculator_shortage
main_screen_drawer
main_screen_empty
main_screen_error
main_screen_item_selection
main_screen_item_selection_failed
main_screen_item_selection_loading
main_screen_store_selection
main_screen_store_selection_failed
main_screen_store_selection_loading
settings_screen
settings_screen_install_not_permitted
settings_screen_up_to_date
settings_screen_update_available
settings_screen_update_check_failed
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
Nexus 7 の横向きで電卓ダイアログが画面からはみ出し、もどる・かいけいのボタンがスクロール領域の中に隠れて押せなくなっていた。あわせて、おつり画面でかいけいを押してからレシートが出るまで数秒かかるのに、画面上は何も変わらず二重に押せてしまう状態だった。
変更内容
電卓ダイアログのレイアウト
会計中の進捗表示
テスト
Generated by Claude Code