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

待機曲一覧画面の作成 #15 #25

Open
wants to merge 21 commits into
base: development
Choose a base branch
from
Open

Conversation

tadasho
Copy link

@tadasho tadasho commented Jul 29, 2017

close #15
待機曲一覧画面を画像・テキストを配置したので、レビューをお願いします。

Copy link

@mori-atsushi mori-atsushi left a comment

Choose a reason for hiding this comment

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

  1. 要素を入れ子構造にすること
    back > next > wide > next-song
  2. 背景画像(back等)ImageではなくPanelコンポーネントを使い、その中に中身の要素を入れること
  3. GameObject名を全体的に再検討すること
    頭は大文字、12345等意味の分からない名前、wide等意味が広すぎる名前はやめる
  4. 画像は9sliceを使用し画像をできるだけ小さくした方がいいかも?(これについては若干不安あり)
    http://qiita.com/lycoris102/items/98bc5a5659e4889dd43f

@mori-atsushi
Copy link

@meokz 大峠さんからもチェックお願いします。

Copy link
Contributor

@meokz meokz left a comment

Choose a reason for hiding this comment

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

@mori-atsushi が書いているように,GameObjectを入れ子にした方が良いですね。

今回は配置するだけですが,曲のところは将来リストにする必要がありそうです。リスト化はissueにあげますか? @mori-atsushi

@mori-atsushi
Copy link

mori-atsushi commented Aug 3, 2017

リストはスクロールビューを使うのかな?
可能ならもうやっちゃったほうがいい気がします。

画像を9sliceを使用し小さくするべきかどうか気になります。

@meokz
Copy link
Contributor

meokz commented Aug 3, 2017

スクロールビューを使えばできますね。
スクロールビューについてはこの辺を参考に。
http://tsubakit1.hateblo.jp/entry/2014/12/18/040252
1回これだけmergeしちゃってissue立て直すでも大丈夫です。

後先考えると9slice使うべきですね

@tadasho
Copy link
Author

tadasho commented Aug 5, 2017

要素の入れ子構造・Panel・名称変更 について修正しました。
レビューお願いいたします。
9sliceとスクルールビューもこのイシューでやってしまいますか?
それとも分けますか?

@mori-atsushi
Copy link

9sliceはそんなに難しくないのでやっちゃいましょう
スクルールビューはやったことがないのでわからないんだけど、ちょっと大変かな?

@tadasho
Copy link
Author

tadasho commented Aug 5, 2017

とりあえず9sliceに手をつけますね!

@tadasho
Copy link
Author

tadasho commented Aug 5, 2017

9sliceの設定をし、画像をまとめました。
レビューをお願いします!

Copy link

@mori-atsushi mori-atsushi left a comment

Choose a reason for hiding this comment

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

  1. circle.pngは他のScneでも使用してるので、
    Assets/Material/circle.png に配置してください。また、ExcitementChart.unityがちゃんと表示されていることを確認してください。
  2. 9sliceを使用すると画像はもっと小さくて済むはずなので、余分な部分はトリミングしてください。
  3. 画像名もわかりやすく変更してください(Unity上でファイル名を変更すること)
  4. Text関係はstretchを使いできるだけwidthとheightを大きくし、Alignmentで位置調整をしてください。
  5. 全ての欄に曲名、アーティストを仮記述してください。
  6. 階層構造の親子関係をもう少し考慮してください。曲番号の子要素に曲名が入ってるのには違和感があります。
  • Panel
    • NexSontFrame
      • NextSongCircle(現NextSong)
        • NextSongWord
      • Song01
      • Artist01
      • LengthLine

@tadasho
Copy link
Author

tadasho commented Aug 8, 2017

レビューの1,3,4,5,6について対応しました。
2の画像をトリミングして小さくする件ですが、僕の技量では画像の処理ができませんでした><
その点のみ他の方に対応をお願いしたいです。

Copy link

@mori-atsushi mori-atsushi left a comment

Choose a reason for hiding this comment

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

  1. 文字はここでstrechに設定してください。

2017-08-08 15 32 24

  1. 階層構造を最初から全部提示したほうが良かったですね… 透明panelを作成し、1曲ずつまとめてください。
  • SongPanel1
    • SongTitle
    • SongArtist
    • LengthLine
    • Underline
    • Circle
      • Number
  • SongPanel2
    • SongTitle
    • SongArtist
    • LengthLine
    • Circle
  1. 縦の線の位置を合わせてください。あと、画像もデザイン案と異なりますね。

2017-08-08 15 36 48

  1. 全体的に位置が適当なので再確認お願いします。
    例えば、SongList(予約曲一覧の文字)はleft、rightの値は0にしてください。その他、親との関係を意識しながら、strechにすべきかも含めて調整してください。

  2. 画像はslackで渡します。

@mori-atsushi
Copy link

引き継いで修正しました。 @meokz さんレビューお願いします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

待機曲一覧画面の作成
3 participants