Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

フロントUIの改善 #41

Merged
merged 27 commits into from
Nov 16, 2024
Merged

フロントUIの改善 #41

merged 27 commits into from
Nov 16, 2024

Conversation

YutaK1026
Copy link
Collaborator

#40 に紐づく改善を行いました。

Copy link

vercel bot commented Nov 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
oricube ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 16, 2024 5:43am

Copy link
Collaborator

@Riku-0801 Riku-0801 left a comment

Choose a reason for hiding this comment

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

ありがとう!いくつか気になったところをコメントしてます!

  • ログインボタン
    • 矢印の向き逆じゃない?
    • 左向きに矢印が向いてるのがちょっと違和感ある気がした。
      • ログアウトならわかる
  • 検索ボックス
    • バツの主張が激しすぎるように思った
    • 色がくろすぎ? 灰色にするとよい?
    • とりあえず虫眼鏡マークよりも大きくて濃くて目立つのはどうかな?って思った
  • リピートボタン
    • 発想はいいと思ったんだけど、有効、無効のときの色が分かりづらいかも
      • 黒と藍色は差が分かりづらい(灰色と藍色とかは?)
    • あともうちょっと太くてもいいかも
  • ページネーション部分
    • 仕方ないけど段階によって幅が変わるのはちょっと気になる
    • トラックの手順4のときとか
  • ホーム画面
    • わかりやすくなってる気がする
    • これ影で浮かせるんじゃなくて文字部分のバックグラウンドカラーを白にしたらどんな感じになるのか気になる

@YutaK1026
Copy link
Collaborator Author

@Riku-0801
ありがと!
対面ならまだしも、チャットでレビューするときはやっぱりfigmaあった方が返しやすいね
ちょっとfigmaの移植とか工夫してみます

@YutaK1026
Copy link
Collaborator Author

YutaK1026 commented Nov 12, 2024

ゆってぃーのコメント

  • ステップ数、場所が微妙。
     - タイトルと説明の部分。いまおってる部分が「よっとの⚪︎手目だよ」というふうにした方がわかりやすい
     - PCでも表示させたい
  • ページネーションは、広がらないように実装したい
  • すぐにループ再生されるのが嫌だ。数秒待って欲しい。
     - 再生ボタン自体が、ループ待機時間のプログレスバーになって欲しい(figmaに書く)
     - 現在の形は崩さなくていい。再生ボタンだけ変更。
  • ログインボタン、googleでログイン、に変更した方がいい、というか規約がそうだった。
  • ボタンのホバーが変。size=28, width, height = 44で。
  • 検索バーの高さを高くしたい

@YutaK1026
Copy link
Collaborator Author

ループ停止
 アイコンをループボタンに斜線が入ったのにする。iconになければ作る

@YutaK1026
Copy link
Collaborator Author

YutaK1026 commented Nov 12, 2024

スクリーンショット 2024-11-12 21 43 02

これの、左右の...を消す方針でいく

@YutaK1026
Copy link
Collaborator Author

スクリーンショット 2024-11-12 21 44 21

この時の1,2の間の...はいらない

@YutaK1026 YutaK1026 merged commit 96b31ea into main Nov 16, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants