Skip to content

Add support to regular expressions using a global parameter. - #61

Open
bram-ontoforce wants to merge 4 commits into
helior:masterfrom
ontoforce:master
Open

Add support to regular expressions using a global parameter.#61
bram-ontoforce wants to merge 4 commits into
helior:masterfrom
ontoforce:master

Conversation

@bram-ontoforce

Copy link
Copy Markdown

This PR adds support to regular expressions using a global parameter e.g.

<Highlight search=/brown|jumps/g">The quick brown fox jumps over the lazy dog</Highlight>

will both highlight 'brown' and 'jumps'. This handles issue #60.

@yeou

yeou commented Oct 11, 2018

Copy link
Copy Markdown

@helior Any news on this PR?

@mihaiserban

Copy link
Copy Markdown

can this branch be merged? support for multiple highlights using regex would be great to have.

@mihaiblaga89

Copy link
Copy Markdown

I'm wondering why this PR adds request as a dependency.

@bram-ontoforce

Copy link
Copy Markdown
Author

I'm wondering why this PR adds request as a dependency.

I added request to the devDependencies to make the tests run. I saw that another PR was opened proposing this change (#58) so I'll remove the package from this branch.

This reverts commit b61b83f.
@kevinmacek

Copy link
Copy Markdown

This would be great to have.

@danperrout

Copy link
Copy Markdown

This would be great to have.

Indeed!

@ahong1

ahong1 commented Nov 5, 2019

Copy link
Copy Markdown

@helior please merge this in. This is a great addition!

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.

7 participants