-
Notifications
You must be signed in to change notification settings - Fork 0
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
メイン画面の画像の一覧表示 #45
base: develop
Are you sure you want to change the base?
メイン画面の画像の一覧表示 #45
Conversation
モック用のjpgファイルまでpushしています... |
{ | ||
"images" : [ | ||
{ | ||
"filename" : "copper_wire_rack.jpg", | ||
"idiom" : "universal", | ||
"scale" : "1x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "2x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "3x" | ||
} | ||
], | ||
"info" : { | ||
"author" : "xcode", | ||
"version" : 1 | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この辺のやつ全部ローカルからimportしたときに出るやつなので削除お願いします
{ | ||
"images" : [ | ||
{ | ||
"filename" : "garden_strand.jpg", | ||
"idiom" : "universal", | ||
"scale" : "1x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "2x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "3x" | ||
} | ||
], | ||
"info" : { | ||
"author" : "xcode", | ||
"version" : 1 | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここも
{ | ||
"images" : [ | ||
{ | ||
"filename" : "gatsby_hat.jpg", | ||
"idiom" : "universal", | ||
"scale" : "1x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "2x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "3x" | ||
} | ||
], | ||
"info" : { | ||
"author" : "xcode", | ||
"version" : 1 | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここも
{ | ||
"images" : [ | ||
{ | ||
"filename" : "gilt_desk_trio.jpg", | ||
"idiom" : "universal", | ||
"scale" : "1x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "2x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "3x" | ||
} | ||
], | ||
"info" : { | ||
"author" : "xcode", | ||
"version" : 1 | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここも
{ | ||
"images" : [ | ||
{ | ||
"filename" : "shrug_bag.jpg", | ||
"idiom" : "universal", | ||
"scale" : "1x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "2x" | ||
}, | ||
{ | ||
"idiom" : "universal", | ||
"scale" : "3x" | ||
} | ||
], | ||
"info" : { | ||
"author" : "xcode", | ||
"version" : 1 | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここも
.cornerRadius(20) | ||
|
||
VStack(alignment: .leading, spacing: 4) { | ||
Text(title).font(.system(size: 14)).padding(.leading, 10) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
コンテンツのとこもそうだけど、だいぶ前に作ったatomのtextから引用しないとアプリ内文字に統一感なくなるのでお願いします
@@ -9,12 +9,12 @@ import SwiftUI | |||
|
|||
struct FloatingActionButtonView: View { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここらへんのファイルもupしないで欲しいです
Text(content).font(.system(size: 14)).padding(.leading, 10) | ||
} | ||
.frame(width: 150, height: 50, alignment: .leading) | ||
.background(Color.white) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
backgroundの色がfigmaとちがう
let data: [CustomCell] = [ | ||
CustomCell(title: "garden_strand", content: "ネックレス", imgPath: "garden_strand"), | ||
CustomCell(title: "gatsby_hat", content: "ハット", imgPath: "gatsby_hat"), | ||
CustomCell(title: "stella_sunglasses", content: "グラス", imgPath: "stella_sunglasses"), | ||
CustomCell(title: "strut_earrings", content: "イヤリング", imgPath: "strut_earrings"), | ||
CustomCell(title: "vagabond_sack", content: "リュックサック", imgPath: "vagabond_sack"), | ||
CustomCell(title: "varsity_socks", content: "ソックス", imgPath: "varsity_socks"), | ||
CustomCell(title: "whitey_belt", content: "ベルト", imgPath: "whitey_belt"), | ||
CustomCell(title: "copper_wire_rack", content: "ラック", imgPath: "copper_wire_rack"), | ||
CustomCell(title: "gilt_desk_trio", content: "小物入れ", imgPath: "gilt_desk_trio"), | ||
CustomCell(title: "shrug_bag", content: "バッグ", imgPath: "shrug_bag") | ||
|
||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
プレビューは下の CollectionView_Previews: PreviewProvider に書いてほしい
.frame(width: 150, height: 152) | ||
.overlay( | ||
RoundedRectangle(cornerRadius: 20) | ||
.stroke(Color.gray, lineWidth: 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
figma上では枠線がないような気がする
白にするのが良いかと
issue #11
機能説明
ex.) hoge機能の実装をしました、huga機能の修正をしました などを箇条書きで記述
懸念事項
ex.) iPhoneSEだとUIが崩れる、swiftlintで注意が出る などの懸念と対策を箇条書きで記述
スクリーンショット
セルフレビュー
ex.)
(あれば)レビュー,チェックしてほしい部分
ex.)