Hi @AlexzundeR welcome to the project!

This feature looks interesting and we'd love to take this further. I have a few points:

1. Unit tests - I didn't see any unit tests in this pull request. We have comprehensive test coverage and want to keep coverage high so would need tests for this new feature.
2. There is no need for HighlightCriteria to implement ICriteria (that's for query and filter predicates) - probably worth merging HighlightConfig into the class rather than having a separate one after that.

I'll check out the code a bit more this weekend - let me know if you need any clarification on these points.

Thanks!

Hi @damieng !

1. I added some tests, I hope they will help to understand how working logic of highlighting.
2. I understood you. There was realy no need in HighlightCriteria, and now there is no HighlightCriteria class in project =) All usings of HighlightCriteria changed to HighlightConfig.

I added functionality of chain calling of my extention, to add many properties to highlighting to request.

Thank for feedback=)
I'll wait for your checkout!

I added some tests for Materializers.
Now I think that my Highlight materializer is not materializer in fact, it is more like decorator for response, which changes it before all materializers start process response.