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

メイン画面の画像の一覧表示 #45

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

massu116
Copy link
Contributor

@massu116 massu116 commented Dec 5, 2022

issue #11

機能説明

ex.) hoge機能の実装をしました、huga機能の修正をしました などを箇条書きで記述

  • メイン画面で表示する画像一覧表示の機能を実装しました
  • 5/30更新
    • タップした時のアクションを追加しました(現在はタップしたアイテムをprintしただけ)
    • 枠線の色を修正
    • テキストのフォント, colorを変更
    • PreviewProviderからのデータ入力に修正
    • Assetsから画像を削除

懸念事項

ex.) iPhoneSEだとUIが崩れる、swiftlintで注意が出る などの懸念と対策を箇条書きで記述

  • LazyVGridでセンター寄せにしているせいか,iPhoneSEだとUIが崩れます(おそらくiPadでも崩れる)

スクリーンショット

開発した部分のスクリーンショット iPhone14 Pro Max iPhone 14 iPhoneSE(3st)
スクリーンショット 2023-05-30 18 31 47 | スクリーンショット 2023-05-30 18 32 13 | スクリーンショット 2023-05-30 18 30 55 |

セルフレビュー

ex.)

  • 画像をリストで渡したときに全て表示されることを確認しました
  • タップした時に正しいアイテム名がログに出力されることを確認しました。

(あれば)レビュー,チェックしてほしい部分

ex.)

  • 表示に関して,画面サイズに合わせてpaddingを入れる方法を検討したいです

@KeitaOsaki KeitaOsaki requested a review from mrs1669 January 29, 2023 17:37
@zekuta-x
Copy link
Contributor

zekuta-x commented Feb 17, 2023

モック用のjpgファイルまでpushしています...
vscode とか使ってcommit するとミス減りますよ

Comment on lines 1 to 21
{
"images" : [
{
"filename" : "copper_wire_rack.jpg",
"idiom" : "universal",
"scale" : "1x"
},
{
"idiom" : "universal",
"scale" : "2x"
},
{
"idiom" : "universal",
"scale" : "3x"
}
],
"info" : {
"author" : "xcode",
"version" : 1
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

この辺のやつ全部ローカルからimportしたときに出るやつなので削除お願いします

Comment on lines 1 to 21
{
"images" : [
{
"filename" : "garden_strand.jpg",
"idiom" : "universal",
"scale" : "1x"
},
{
"idiom" : "universal",
"scale" : "2x"
},
{
"idiom" : "universal",
"scale" : "3x"
}
],
"info" : {
"author" : "xcode",
"version" : 1
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

ここも

Comment on lines 1 to 21
{
"images" : [
{
"filename" : "gatsby_hat.jpg",
"idiom" : "universal",
"scale" : "1x"
},
{
"idiom" : "universal",
"scale" : "2x"
},
{
"idiom" : "universal",
"scale" : "3x"
}
],
"info" : {
"author" : "xcode",
"version" : 1
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

ここも

Comment on lines 1 to 21
{
"images" : [
{
"filename" : "gilt_desk_trio.jpg",
"idiom" : "universal",
"scale" : "1x"
},
{
"idiom" : "universal",
"scale" : "2x"
},
{
"idiom" : "universal",
"scale" : "3x"
}
],
"info" : {
"author" : "xcode",
"version" : 1
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

ここも

Comment on lines 1 to 21
{
"images" : [
{
"filename" : "shrug_bag.jpg",
"idiom" : "universal",
"scale" : "1x"
},
{
"idiom" : "universal",
"scale" : "2x"
},
{
"idiom" : "universal",
"scale" : "3x"
}
],
"info" : {
"author" : "xcode",
"version" : 1
}
}
Copy link
Contributor

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)
Copy link
Contributor

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 {
Copy link
Contributor

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)
Copy link
Contributor

Choose a reason for hiding this comment

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

backgroundの色がfigmaとちがう

Comment on lines 46 to 58
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")

]
Copy link
Contributor

Choose a reason for hiding this comment

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

プレビューは下の CollectionView_Previews: PreviewProvider に書いてほしい

Comment on lines 33 to 36
.frame(width: 150, height: 152)
.overlay(
RoundedRectangle(cornerRadius: 20)
.stroke(Color.gray, lineWidth: 1)
Copy link
Contributor

Choose a reason for hiding this comment

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

figma上では枠線がないような気がする
白にするのが良いかと

@massu116 massu116 mentioned this pull request Jun 5, 2023
6 tasks
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