Closed yasyun closed 3 years ago
TagDetailですが モーダル以外にも違う点がありそうです。
issueを拝見しましたが、
- 制約修正により崩れなくなった → しかしセルの中で制約が壊れてしまった。いろいろ試しましたが改善できなかったので、アドバイスをいただきたいです。
どのような修正行ったのか? 何を試したのか? をissueにのせて欲しかったです。
あと https://youtu.be/hZwQBF8gUmQ ↑こちらを見ましたか? わかりづらい点あったら教えてください。
issueを拝見しましたが、
- 制約修正により崩れなくなった → しかしセルの中で制約が壊れてしまった。いろいろ試しましたが改善できなかったので、アドバイスをいただきたいです。
どのような修正行ったのか? 何を試したのか? をissueにのせて欲しかったです。
あと https://youtu.be/hZwQBF8gUmQ ↑こちらを見ましたか? わかりづらい点あったら教えてください。
issueの方にこれまでの修正履歴を覚えている限りですが記載しました。(issue #3) shuさんの解説動画は拝見しました。動画の内容はとても分かりやすかったと思います、ありがとうございます。 動画内容を試した上でうまくいかず、Pull Requestを出したのですが、改善案を思いついたので少しそちらを試してみようと思います。 その上で動画の内容について疑問点があれば、折り返し質問させていただこうと思います。よろしくお願いします。
制約の問題は解決できました。ありがとうございました。
タグ画面のアイコンがキャッシュ化できていないので、次回にその点も併せて修正します。
dateFormatメソッドを修正したことで、FeedPageのセルにおける表示の部分のコードにも差分が出ています。また、タグ画面のアイコンもキャッシュ化しました。
赤枠内のスペースが気になりました。 別のブランチで修正予定でしたら、保留で構いません。 LGTMです。
コメントしましたので確認お願いします。
画像のキャッシュ化対策は早めにやっておいたほうが良いと思います (別issue)
あと、iPhone12 pro maxでビルドするとレイアウト描画おかしいのでチェックしておいてください。
このレイアウトが崩れてしまう現象については、セルの大きさがたまに変化してしまっている(おそらくLabelが悪さをしている)ためだと見受けたので、別でissueを作りそちらで修正してみようと思います。 issueはこちらです。(CollectionViewのCellレイアウト)
SearchBarのクラッシュエラーに対する修正をしたので、FeedPageViewController.swiftにも差分が出ています。 issueはこちらです。(SearchBarクラッシュエラー)
issue #3
レビューして欲しい点 TagListPageViewController.swift, TagListCell.swift, TagDetailPageViewController.swift, TagDetailCell.swiftを中心に見ていただきたいです。
その他 ・issueにも書いてあるのですが、CollectionViewの制約が壊れてしまっていて、その点でアドバイスをいただきたいので、よろしくお願いいたします。 ・FeedPageにも差分が出ていると思いますが、セルをタップして記事を表示する時はお手本を見るとモーダル遷移のようだったので、その点のみ修正しました。 ・後から気付いたのですが、タグの詳細ページに移動する際にモーダル遷移になっており、このままだと元の画面に戻れないので、後ほどNavigationControllerで実装し直します。ひとまず最初はスルーでお願いします。 → 修正しました。