Closed Rhime-design closed 5 years ago
Android版もですが、Hello NEMber、やっぱ別に要らないかな…いや…どうなんだろう
何にします?消しちゃうだけだと寂しい?そんなことない? と思って色々見てみたけどTopのToolbarって結構みんな持て余してますね ハンバーガー(うちでいうNEMボタン)とかは置いてあるけどテキストスペースはみんな微妙な感じ
Spotifyも HOME とだけ いらない情報 YoutubeもロゴとYoutube Twitter ホーム Kayash ウォレット Amazon ロゴのみ
んー、なんというか内輪な感じがして。ジャーゴン感。 コミュニティ感があるといえばそれはそれで良いのですが。 スカスカになる点に関しては、今後マルチシグ通知とかでベルマークとか置いたりも可能性としてあるので、まあいいかなという。
なるほど。そしたら無難にRaccoonWallet とかにしちゃったほうがいいか。 Rhimeさんの納得いく感じに調整してもらって問題ないですが大きくいじるのは通知とか回りの実装をしたときですかね(トランザクション通知 ハーベスト通知 マルチシグ 等) ベル置いたらNEMマークも移動しそうだし
やっぱツールは右で、ドロワーは左ですかね? なんか世間と感覚が逆?wドロワー右のほうが自然な気がするんですが、よく分からぬ。
←🔔 [真ん中:ロゴ] ⭐️ドロワー
みたいなのでもいいかなとか。 そのあたりは時がきたらデザインのほうで置いて確認してみます。
基本的にはそうかと思います笑 あくまで私のは根拠のない一般的なアプリにおいてはそう、という視点なのでアレですが。
🍔 RaccoonWallet 🔔  ̄ ̄ ̄ ̄ ̄ ̄ ̄ ̄ ̄ ̄ ̄ ̄ みたいなののほうがよく見る感じではあります。
そうですね!通知も一応ブランチはあるっちゃあるのでまた近くなってきたら相談します!
レビューありがとうございます!
ざっと確認した感じだいたい修正できそうなんですが、最初の
Android版みたいにHOMEをスワイプでシュッシュッっと移動したい。
に関しては標準のボトムナビゲーションの UI だと対応できないのでかなり難しいです。 App Store とかブック、ヘルスケアなどプリインストールされてる iOS アプリでもスワイプで切り替わらないので、対応が必須なものではないと思ってるのですが、やっぱり対応した方が良いですかね?
あと、
送金確定時にPINを使う際のボタンがない。
すみません、PWA 版もこの実装だったのですが、 「準備は良いですか?指紋認証で送信します」 みたいな表示がされている部分が全体タップできるようになっていて、そこで PIN入力(または生体認証)のダイアログが表示されます。
Android のようにバックグラウンドで指紋認証を待機するようなことはできないので、指紋認証で送信する時も PIN 入力のダイアログを表示する必要があります。
不具合のところにある「指紋認証で送金できない?」の部分もその話ですかね。
iOSリポジトリinviteしてもらったので、やり取りはあちらでやることにしますね!
▼UI/UX
[ ] Android版みたいにHOMEをスワイプでシュッシュッっと移動したい。
[ ] 左端→→スワイプでドロワーを出したい。
[ ] トランザクション画面でのスクロール時に、heroimageが隠れない
[ ] Android版もですが、Hello NEMber、やっぱ別に要らないかな…いや…どうなんだろう
[ ] Android版もですが、ウォレットバーのウォレットボタンのタップ反応範囲がシビアなので、ボタンより一回り大きくしたい。
[ ] アドレスコピー(他)のトーストがボトムナビに被るのをどうにかしたい👀 ボトムナビはドロワーを除けば最上位Layerに位置していて、他の要素侵食されないでほしい。
[ ] HOMEの残高やトランザクションの小窓が表示のつどローディングされるのを、初回で取得した後、キャッシュとか(?)を使ってイベント時やユーザーの下スワイプが入力されたとき以外は静的に表示しておいてほしい。
[ ] ローディングが要素ごとに表示される部分に関して。 ↑に書いた挙動を前提にしていますが、 iOS標準(たぶん)のちっこいローディングを、オーバーレイで画面中央に表示する程度で良いと思いました。![2018-09-27_09h39_28](https://user-images.githubusercontent.com/34479640/46116896-3ef57c00-c239-11e8-9044-6b29ac30fa98.png)
[ ] 送金確定時にPINを使う際のボタンがない。
▼不具合・改善
まだざっくりなんで、随時追記します。