模擬案件_#8_addUserPage

[yasyun]

• issue: #8
• レビューして欲しい点 ・Userフォルダの中にあるUserPageViewControllerとUserPageCellをレビューしていただきたいです。
• その他 ・基本的な実装内容は、MyPage(#4)と同じです。

[ShuheiYoshidaJP]

• UserPageViewController.swift
• UserPageCell.swift

↑このファイルプッシュしましたか？

[yasyun]

ユーザーページに進むとタブバーが消えますね。 修正お願いします。

Figmaの完成例の方ではUserPageにタブバーが表示されていなかったのでわざと消しているんですが、それでもタブバーは残しておいた方がよいでしょうか？

[ShuheiYoshidaJP]

ほんとですね！ タブバーが消えてました😳 LGTMです。

[Kaito-a-bit]

・「投稿記事」のヘッダーが描出されていないので、その部分の修正をお願いします。

[Kaito-a-bit]

フォントボールドではないので修正お願いします。 またヘッダーの背景色がスクロール開始まで白くなってしまっているので、その部分についても修正お願いします。 （TableViewの上に恐らくスペースが入っている？ように見えます。）

https://user-images.githubusercontent.com/73689418/152665691-453015a9-5413-4ff2-9002-82eddda69168.mov

[yasyun]

ヘッダーの背景色を設定していなかったので、設定してあげたらスクロール開始前でも色がつきました。 MyPageとUserPageどちらにも同様の修正を行いました。 また、UserPageからFollowPageに戻ったときにタブバーが消えたままになってしまっていたので、FollowPageにviewWillAppearを追加して、タブバーが再出現するように修正しました。

[Kaito-a-bit]

LGTMです👍ちなみにTableViewのheaderの上のマージンの正体って分かりましたか？

[yasyun]

LGTMです👍ちなみにTableViewのheaderの上のマージンの正体って分かりましたか？

確認してみたところTableViewのheaderの上部分は特に何も空いていないように感じました。 他のTableViewも同じようになっているので、デフォルトの状態のままだと思います。 ただ、まだheaderの太さをこちらで28ptというように指定してあげていないので、そのせいでheaderの上にマージンが入ってしまっている？のかなとか思いました。

[yasyun]

あの謎のマージンの正体はTableHeaderViewだったみたいです。 僕が実装したときに、ヘッダー部分でTableHeaderViewではなくSectionHeaderを使用していたため、空白のTableHeaderViewが上に残り、謎の隙間になってしまっていたみたいです。 そのため、MyPageとUserPageどちらとも、SectionHeaderではなくTableHeaderViewを使用する形に変更したところ、この問題は解決できました。

[Kaito-a-bit]

LGTM