uhooi / UhooiPicBook

Uhooi's character book.

MIT License

108 stars 25 forks source link

Feature/spotlight #24

Closed Frog-Frog closed 4 years ago

Frog-Frog commented 4 years ago

20 やったこと

・MonsterList画面で選択時にSpotlight検索とUserDefaultsにモンスターを保存・Spotlightで開かれた時に、UserDefaultsからモンスターを取り出し、MonsterDetail画面へ遷移

Preview

UhooiSpotlight

uhooi commented 4 years ago

@Frog-Frog 神PRありがとうございます🙏 1つ気になったのが、 MonsterList の表示時に全モンスターを保存せず、 MonsteraDetail の表示時に一体しか保存していないことです！一度開いたモンスターが履歴として確認できるのが、よりSpotlight的な使い方ということでしょうか？？

細かくは明日レビューさせていただきます🙏

Frog-Frog commented 4 years ago

@uhooi Safariなどでは検索の履歴を表示しているので、UhooiPicBookでも詳細の閲覧履歴を検索させたかったという意図が第一にあります。

それに加えて、MonsterListで取得した全モンスターの保存という形にしてしまうと、今後モンスターが何かのタイミングで爆増した際に、Spotlightへ一度に大量の保存をすることでアプリに負荷がかかってしまうと申し訳ないからです😂

uhooi commented 4 years ago

@Frog-Frog なるほど、パフォーマンスまで考えていただきありがとうございますー！仕様はこちらで問題ありませんね！

uhooi commented 4 years ago

@Frog-Frog 迅速なご対応ありがとうございます😭 完了したらメンション付きで報告してくれると嬉しいです！一旦マージして、自分が直すと言ったところはマージ後に改めて直しますー

ref: https://twitter.com/giginet/status/1260093759716392960?s=20

Frog-Frog commented 4 years ago

@uhooi 一旦全て対応できたかと思います！

Allow edits by maintainersもチェック入れてるので、あとはお任せします👍

uhooi commented 4 years ago

@Frog-Frog ありがとうございますー！マージしました！修正は別ブランチで行います💪