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

Fix #1660: Search as you type #1826

Merged
merged 7 commits into from
Dec 17, 2018
Merged

Fix #1660: Search as you type #1826

merged 7 commits into from
Dec 17, 2018

Conversation

nojvek
Copy link
Contributor

@nojvek nojvek commented Dec 10, 2018

  • Add incremental option to ISearchOptions
  • DRY examples page
  • Improve performance of translateBufferLineToStringWithWrap with lineCache

Fixes #1660

@Tyriar Tyriar added this to the 3.10.0 milestone Dec 11, 2018
@nojvek nojvek mentioned this pull request Dec 11, 2018
@Tyriar
Copy link
Member

Tyriar commented Dec 14, 2018

@nojvek I'm happy to go forward with this without tests unless you had a good idea of how to test it. I had a look and it's a bit tricky as right now none of the tests really involve TerminalCore/SelectionManager as they are testing findInLine not findNext/Previous. Addon tests should be made easier as part of #1128 when we also pull these out of the core repo.

@nojvek
Copy link
Contributor Author

nojvek commented Dec 15, 2018 via email

@Tyriar Tyriar merged commit a53c6f1 into xtermjs:master Dec 17, 2018
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