Adding topic-list-after-title outlet to the mobile template

Dev
1

In the events plugin, we currently override the mobile template. I’m very appreciative of the recent outlet additions to the mobile template which already make things a lot better. Just wondering whether we can have topic-list-after-title also on mobile so we can simply get rid of our override? This outlet is already available on desktop btw.

I’m happy to make a PR for this if the request makes sense.

4 Likes
2

Sure, I think that sounds reasonable - I can’t think of any reason to have it on missing on mobile.

:heart_eyes: :heart_eyes:

5 Likes
3

https://github.com/discourse/discourse/blob/45eb97c2029a27455340809428ae4572b6731b8b/app/assets/javascripts/discourse/app/templates/mobile/list/topic-list-item.hbr#L25

Is this the right place to add it? Just confirming.

1 Like
4

:+1: that seems to line up nicely with the desktop template

1 Like
5

Here’s the PR for this. https://github.com/discourse/discourse/pull/10059

3 Likes
6

@david thanks for the merge.

3 Likes
7

This topic was automatically closed after 7 hours. New replies are no longer allowed.