flarum / framework

Simple forum software for building great communities.
http://flarum.org/
6.39k stars 834 forks source link

[1.x] [tags] feat: option to display tags in DiscussionsSearchSource #4055

Closed imorland closed 1 month ago

imorland commented 1 month ago

Changes proposed in this pull request: Introduces an option to display the tag label(s) above the discussion title in search results (DiscussionsSearchSource). Disabled by default to maintain current behaviour.

Reviewers should focus on: @flarum/core I feel this is something to also include in 2.x?

Screenshot Search: search

Tags settings: tags settings

QA

Necessity

Confirmed

Required changes:

SychO9 commented 1 month ago

Why not just make a custom extension? especially since this is not something that can be ported to 2.x as 2.x search UI is highly different and has tags by default

imorland commented 1 month ago

Why not just make a custom extension? especially since this is not something that can be ported to 2.x as 2.x search UI is highly different and has tags by default

Fair comment, I've not seen 2.x for some time, so didn't know about that!

Yes a custom extension can absolutely be created now that DiscussionsSearchSource and DiscussionsSearchItem can be extended. The feeling was to push this functionality back to the root so that it can potentially benefit the whole 1.x ecosystem :)

SychO9 commented 1 month ago

imho, a custom extension released to the wider community is just as beneficial for those looking for such a feature.

imorland commented 1 month ago

imho, a custom extension released to the wider community is just as beneficial for those looking for such a feature.

Fair enough, I'm having issues with typings (again!) on this one anyway!. Will close this and consider a standalone extension :)