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パーツの配置とAssetsのImage追加 #109

Closed
wants to merge 4 commits into from

Conversation

eyener3
Copy link
Contributor

@eyener3 eyener3 commented May 5, 2019

fixes #87

(#の後にcloseしたいissueの番号を記述してください)

Summary(要約)

  • StoryboardへUIパーツの配置

  • AssetsへのImageを追加

Other Information(他の情報)

  • アクションシートとStackViewは別Issueで行なう

Tested(テストしたこと)

  • シュミレーターが立ち上がらないのでテストできず...

Must Reviewer(必須レビュアー)

@riscait

スクリーンショット 2019-05-06 05 17 26

@eyener3 eyener3 added the enhancement 新しい機能あるいはそのリクエストです label May 5, 2019
@eyener3 eyener3 added this to the v1.0.0 milestone May 5, 2019
@eyener3 eyener3 self-assigned this May 5, 2019
Copy link
Contributor

@riscait riscait left a comment

Choose a reason for hiding this comment

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

配置が目に見えると良いですね✨
ProgressBarとCheckboxは仮の配置で、これから専用のクラスを作っていくって認識で合ってますよね😊?
エラーが出てる際は、エラー文をコピペするのとスクリーンショットがあると助かります👌
2箇所にコメントしました!

@@ -44,7 +44,7 @@

/* Begin PBXFileReference section */
0B04AA79227A9D5100E18F0A /* ProgVC.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ProgVC.swift; sourceTree = "<group>"; };
0B04AA7B227A9DB200E18F0A /* Prog.storyboard */ = {isa = PBXFileReference; lastKnownFileType = file.storyboard; path = Prog.storyboard; sourceTree = "<group>"; };
0B04AA7B227A9DB200E18F0A /* Prog.storyboard */ = {isa = PBXFileReference; lastKnownFileType = file.storyboard; name = Prog.storyboard; path = "/Users/katouayana/Xcode/Stepippo-iOS/Stepippo/Classes/Views/Prog.storyboard"; sourceTree = "<absolute>"; };
Copy link
Contributor

Choose a reason for hiding this comment

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

ここ、あいなーさんのローカルパスになっちゃってますね🤔
LocationをAbsoluteに変更したのが原因かと思います。
他のファイルに合わせて元に戻して、パスも正しましょう👌

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます🙇🏻‍♀️🙏
色々といじっているうちに、
変更してしまっていたようです..><
元に戻して、パスも正せるようにしてみます💡

</accessibility>
<rect key="contentStretch" x="0.0" y="0.0" width="0.5" height="1"/>
</imageView>
<textField opaque="NO" contentMode="scaleToFill" fixedFrame="YES" contentHorizontalAlignment="left" contentVerticalAlignment="center" text="進捗" borderStyle="roundedRect" textAlignment="natural" minimumFontSize="25" translatesAutoresizingMaskIntoConstraints="NO" id="IIC-1P-lHJ">
Copy link
Contributor

@riscait riscait May 6, 2019

Choose a reason for hiding this comment

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

画面タイトルは、NavigationBarのTitleに設定しますのでここではTextFieldは不要ですね👌
MiscVCをみていただければわかりますが、コードで設定する方法とStoryboardで設定する方法の2種類があります。
Storyboardで設定するメリットは、簡単・見れること
コードで設定するメリットは、あとでローカライズ(多言語対応)しやすかったり、条件分岐で可変にすることができるところです👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

詳しく教えていただき、
ありがとうございます^-^!

龍之介さんのMiscVCをもう一度見て、
コードとStoryboardとの設定の違いを、
ちゃんと理解したいと思います🙇🏻‍♀️✨

@eyener3
Copy link
Contributor Author

eyener3 commented May 6, 2019

配置が目に見えると良いですね✨

ありがとうございます😊!

ProgressBarとCheckboxは仮の配置で、これから専用のクラスを作っていくって認識で合ってますよね😊?

はい、その認識で合っています!💡

エラーが出てる際は、エラー文をコピペするのとスクリーンショットがあると助かります👌

ありがとうございますっ
storyboardの作業をはじめてからずっと同じエラーが出ていて、
途方に暮れておりまして..後日うかがおうと思っておりました><。
スクリーンショットとエラー文のコピペ送らせていただきますねっ

@eyener3
Copy link
Contributor Author

eyener3 commented May 6, 2019

  • ローカルパスの修正を行ないました

  • シュミレーター時のエラー文
    framework not found Realm
    スクリーンショット 2019-05-06 20 44 27

  • TextField「進捗」を消しました

よろしくお願いします^-^💡

@eyener3 eyener3 requested a review from riscait May 6, 2019 11:49
@riscait
Copy link
Contributor

riscait commented May 6, 2019

  • シュミレーター時のエラー文
    framework not found Realm

@eyener3 #76 こちらのIssueは完了しましたか?🤔
エラー読むと、Realmフレームワークがないからって言ってますよね、carthage bootstrapコマンドでフレームワークを作れば消えるかと思います👌

@riscait
Copy link
Contributor

riscait commented May 6, 2019

@eyener3 commitがPushされていないようです💡

@eyener3
Copy link
Contributor Author

eyener3 commented May 6, 2019

  • シュミレーター時のエラー文
    framework not found Realm

@eyener3 #76 こちらのIssueは完了しましたか?🤔
エラー読むと、Realmフレームワークがないからって言ってますよね、carthage bootstrapコマンドでフレームワークを作れば消えるかと思います👌

#76 まだできていないのですよね..
それが原因だと思うのですが、
何度、挑戦しても上手くいかずで..
近日中に教えていただこうと思っておりました🙇🏻‍♀️🙏

pushしておきますね!💡
ありがとうございます😊

@eyener3
Copy link
Contributor Author

eyener3 commented May 7, 2019

pushしました^-^💡

@eyener3 eyener3 closed this May 8, 2019
@eyener3 eyener3 added this to Done in [あいなー]タスク進捗画面 via automation May 8, 2019
@eyener3 eyener3 deleted the Add_Progstoryboard_UIParts branch May 11, 2019 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 新しい機能あるいはそのリクエストです
Development

Successfully merging this pull request may close these issues.

【進捗画面】Storyboardに必要なパーツを配置する
2 participants