FEATURE: revamping discover page + add hot topics#281
Conversation
pmusaraj
left a comment
pmusaraj
left a comment
There was a problem hiding this comment.
Would be nice to add 1-2 test cases for the new Discover screen partials (tag view, detail view, etc.)
But otherwise, this is great, makes the Discover screen much more dynamic!
| "discover_communities_section": "%{tag} Communities", | ||
| "discover_topics_section": "Recent Discussions in %{tag}", |
There was a problem hiding this comment.
One annoying thing is that we don't translate the tags themselves... so in other languages this will say "communautés gaming" for example... not sure how to fix it. I guess we could add tags as strings individually and that would allow translating them?
There was a problem hiding this comment.
Yeah, or we could have the endpoint in discover(hot-topics-tags.json) return the tag label too, then we could get rid of both:
DiscourseMobile/js/screens/DiscoverScreenComponents/TagSplash.js
Lines 19 to 28 in 07fac6d
and this issue at the same time
There was a problem hiding this comment.
This file has grown quite large, I wonder if we can split it up.
2 participants
See video:
Screen.Recording.2026-04-10.at.18.27.33.mov
Hot topics per tag:
Besides the hot topics list, we have the forum hot topics in their details page