Feeling Lucky
RTC
bug
PR-4.4-dev
[#42921] - [4.4] Smart Search, limit the highlight
- Fixed in Code Base
- 4 Mar 2024
- Medium
- Build: 4.4-dev
- # 42921
- Diff
- Fedik:highlight-limit
User tests: Successful: Unsuccessful:
Pull Request for Issue #42889
Summary of Changes
Same as #41463 but for highlight plugin
Testing Instructions
In testing installation.
In Smart Search config set "Word Match" to "Match words containing the search term anywhere".
Go to search page, and search for joomla 1.
Notice the length of the link for each search result.
Actual result BEFORE applying this Pull Request
it is long
Expected result AFTER applying this Pull Request
it is shorter
Link to documentations
Please select:
- Documentation link for docs.joomla.org:
- No documentation changes for docs.joomla.org needed
- Pull Request link for manual.joomla.org:
- No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
viocassel
- comment
- 29 Feb 2024
I have tested this item ✅ successfully on 84c7fde
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42921.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
bug
PR-4.4-dev
|
||
Quy
- comment
- 29 Feb 2024
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42921.
Hackwar
- comment
- 1 Mar 2024
Sorry, but we have this already in the layout of smart search. https://github.com/joomla/joomla-cms/blob/4.4-dev/components/com_finder/tmpl/search/default_results.php#L64
I don't think it is useful to duplicate this code here as well.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-03-04 15:45:19 |
| Closed_By | ⇒ | laoneo | |
| Labels |
Added:
RTC
|
||
laoneo
- comment
- 4 Mar 2024
Thanks!
I have tested this item ✅ successfully on 84c7fde
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42921.