Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 3a762c3

Browse files
committed
Change search bar UI to always say "Find..." in the first field, instead of
sometimes saying "Find in..." or "Replace in..." (which makes it sound like it's describing the set of files to search in).
1 parent cd78535 commit 3a762c3

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/nls/root/strings.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -133,11 +133,12 @@ define({
133133
"DELETE" : "Delete",
134134
"BUTTON_YES" : "Yes",
135135
"BUTTON_NO" : "No",
136-
136+
137137
// Find, Replace, Find in Files
138138
"FIND_RESULT_COUNT" : "{0} results",
139139
"FIND_RESULT_COUNT_SINGLE" : "1 result",
140140
"FIND_NO_RESULTS" : "No results",
141+
"FIND_QUERY_PLACEHOLDER" : "Find\u2026",
141142
"REPLACE_PLACEHOLDER" : "Replace with\u2026",
142143
"BUTTON_REPLACE_ALL" : "Batch\u2026",
143144
"BUTTON_REPLACE_ALL_IN_FILES" : "Replace\u2026",
@@ -320,7 +321,6 @@ define({
320321
// Search menu commands
321322
"FIND_MENU" : "Find",
322323
"CMD_FIND" : "Find",
323-
"CMD_FIND_FIELD_PLACEHOLDER" : "Find\u2026",
324324
"CMD_FIND_NEXT" : "Find Next",
325325
"CMD_FIND_PREVIOUS" : "Find Previous",
326326
"CMD_FIND_ALL_AND_SELECT" : "Find All and Select",

src/search/FindInFilesUI.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ define(function (require, exports, module) {
150150
multifile: true,
151151
replace: showReplace,
152152
initialQuery: initialString,
153-
queryPlaceholder: (showReplace ? Strings.CMD_REPLACE_IN_SUBTREE : Strings.CMD_FIND_IN_SUBTREE),
153+
queryPlaceholder: Strings.FIND_QUERY_PLACEHOLDER,
154154
scopeLabel: FindUtils.labelForScope(scope)
155155
});
156156
_findBar.open();

src/search/FindReplace.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -569,7 +569,7 @@ define(function (require, exports, module) {
569569
multifile: false,
570570
replace: replace,
571571
initialQuery: initialQuery,
572-
queryPlaceholder: Strings.CMD_FIND_FIELD_PLACEHOLDER
572+
queryPlaceholder: Strings.FIND_QUERY_PLACEHOLDER
573573
});
574574
findBar.open();
575575

0 commit comments

Comments
 (0)