Don
81
これを excluded_categories 設定として実装しました: FEATURE: Add excluded categories setting - Pull Request #13 - VaperinaDEV/discourse-topic-preview-modal - GitHub
当初は、トピックが表示される場所に応じて挙動を変えることを検討していましたが、トピックの実際のカテゴリを確認する方が一貫性があるアプローチだと考えます。
これにより、あるカテゴリが除外されている場合、そのカテゴリ内のトピックは、そのカテゴリ内、/latest、あるいは他のトピック一覧のいずれに表示されていても、モーダルではなく常にフルトピックとして開かれるようになります。
また、同じトピックが場所によってモーダルで開かれることもあれば、フルトピックとして開かれることもあるという、混乱を招く可能性のある状況を回避できるものと思います。
「いいね!」 2
Don
82
2つの問題を修正しました: FIX: unread dot not clearing for nested topics and never-opened topics - Pull Request #14 - VaperinaDEV/discourse-topic-preview-modal - GitHub
- ネストビュー:
readPosts() がモーダル自身の postRegistry ではなく、コアのサイト全体のネストコントローラーに委譲されていたため、モーダル内で投稿ごとの既読状態のドットがクリアされませんでした。
- 新しいトピック:
unread_posts、unread、new_posts のみがリセットされていましたが、新しいトピックのドットを制御する独立した unseen フラグはリセットされていなかったため、トピックをモーダル内で既読にしても、トピックリストに引き続き表示され続けていました。
「いいね!」 1