Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 4 commits into from

Conversation

imorland
Copy link
Member

@imorland imorland commented Oct 2, 2024

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

  • Has the problem that is being solved here been clearly explained?
  • If applicable, have various options for solving this problem been considered?
  • For core PRs, does this need to be in core, or could it be in an extension?
  • Are we willing to maintain this for years / potentially forever?

Confirmed

  • Frontend changes: tested on a local Flarum installation.
  • Backend changes: tests are green (run composer test).
  • Core developer confirmed locally this works as intended.
  • Tests have been added, or are not appropriate here.

Required changes:

  • Related documentation PR: (Remove if irrelevant)
  • Related core extension PRs: (Remove if irrelevant)

@imorland imorland added type/feature javascript Pull requests that update Javascript code labels Oct 2, 2024
@imorland imorland added this to the 1.8.6 milestone Oct 2, 2024
@imorland imorland requested a review from a team as a code owner October 2, 2024 10:36
@SychO9
Copy link
Member

SychO9 commented Oct 2, 2024

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
Copy link
Member Author

imorland commented Oct 2, 2024

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
Copy link
Member

SychO9 commented Oct 2, 2024

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

@imorland
Copy link
Member Author

imorland commented Oct 2, 2024

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 :)

@imorland imorland closed this Oct 2, 2024
@imorland imorland removed this from the 1.8.6 milestone Oct 2, 2024
@imorland imorland removed type/feature javascript Pull requests that update Javascript code labels Oct 2, 2024
@imorland imorland deleted the im/tags branch October 2, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants