Coding-Trip / CodingTripAndroid

CodingTripのAndroidクライアント
https://xd.adobe.com/view/54acd9e3-4978-482e-a0ac-137f89a4e458/
4 stars 0 forks source link

feat:add contributor cell layout #20

Closed morux2 closed 6 years ago

morux2 commented 6 years ago

constraintLayoutでcellを作成

omuomugin commented 6 years ago

@morux2 Android Studio上のpreviewでもいいからスクショおねがいー

morux2 commented 6 years ago
2018-02-14 18 11 24

スクール終わったら確認します!

omuomugin commented 6 years ago

いつでもいいよー

nakker1218 commented 6 years ago

Zeplinに表示されてる大きさが作ったときとだいぶ違うから確認します

omuomugin commented 6 years ago

あれ ++ --のとこどこいったの?

omuomugin commented 6 years ago

○++と○--をセットするtextViewがそれぞれ必要なはず

nakker1218 commented 6 years ago

このコメントで消してたplus, minusのTextView普通に必要なやつだったから復活させよう https://github.com/Coding-Trip/CodingTripAndroid/pull/20/files/8e1d8a739b5f62bc090ed47547739c0868a669f4#r168100135

omuomugin commented 6 years ago

++と--だけのTextViewはいらなくて"3++"とか"5--"を設定できるTextViewが必要っていうのでおっけ?

nakker1218 commented 6 years ago

色変えるのめんどいしそれがいいかなと思うんだけど

omuomugin commented 6 years ago

あ、↑のコメントはそういうつもりだったそもそも。 stringで%d++, %d--みたいに定義してTextViewをそれぞれ1つずつ用意するでいいんじゃないかなって思った。もともとのコード4つTextViewなかったっけと思って。それを2つでよくないっていう感じ。

morux2 commented 6 years ago

結局必要なtextviewっていくつですか?><

nakker1218 commented 6 years ago

commit, plus, minusかな

omuomugin commented 6 years ago

LGTM!

nakker1218 commented 6 years ago

LGTM:+1: