Skip to content

Commit

Permalink
Merge pull request microsoft#146213 from microsoft/rebornix/update-re…
Browse files Browse the repository at this point in the history
…place-all-nb

fix microsoft#142756 update replace all button after input is updated
  • Loading branch information
rebornix authored and Liangcheng Juves committed Mar 31, 2022
2 parents 3c3ea50 + ca1a874 commit f381db0
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ export class NotebookFindWidget extends SimpleFindReplaceWidget implements INote
}

const matches = this._findModel.findMatches;
this._replaceAllBtn.setEnabled(matches.find(match => match.modelMatchCount < match.matches.length) === null);
this._replaceAllBtn.setEnabled(matches.length > 0 && matches.find(match => match.modelMatchCount < match.matches.length) === undefined);

if (e.filters) {
this._findInput.updateFilterState((this._state.filters?.markupPreview ?? false) || (this._state.filters?.codeOutput ?? false));
Expand Down

0 comments on commit f381db0

Please sign in to comment.