Skip to content

Commit d63f228

Browse files
authored
fix: Enable 'High priority' to be selected easily in Auto Suggest (#1996)
Fixes #1943 The order of priorities in auto suggestion is now - high - medium - low - highest - lowest This retains the original order, for people's muscle memory. And it means that the longer names are after the shorter ones, which also makes it possible to type both, and it means that typing parts of names selects the same result as before highest and lowest were added.
1 parent 47b565c commit d63f228

5 files changed

+8
-8
lines changed

docs/Getting Started/Auto-Suggest.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,10 +174,10 @@ Similarly, you can type some fraction of the word `start` (of whatever length is
174174
| Searchable Text | Text that is added |
175175
| ----------------------------- | -------------------------- |
176176
|| \<new line> |
177-
| 🔺 highest priority | 🔺 |
178177
| ⏫ high priority ||
179178
| 🔼 medium priority | 🔼 |
180179
| 🔽 low priority | 🔽 |
180+
| 🔺 highest priority | 🔺 |
181181
| ⏬️ lowest priority | ⏬️ |
182182
| 🔁 recurring (repeat) | 🔁 |
183183
| every | 🔁 every |

src/Suggestor/Suggestor.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,6 @@ function addTaskPropertySuggestions(
8383
appendText: `${symbols.scheduledDateSymbol} `,
8484
});
8585
if (!hasPriority(line)) {
86-
genericSuggestions.push({
87-
displayText: `${symbols.prioritySymbols.Highest} highest priority`,
88-
appendText: `${symbols.prioritySymbols.Highest} `,
89-
});
9086
genericSuggestions.push({
9187
displayText: `${symbols.prioritySymbols.High} high priority`,
9288
appendText: `${symbols.prioritySymbols.High} `,
@@ -99,6 +95,10 @@ function addTaskPropertySuggestions(
9995
displayText: `${symbols.prioritySymbols.Low} low priority`,
10096
appendText: `${symbols.prioritySymbols.Low} `,
10197
});
98+
genericSuggestions.push({
99+
displayText: `${symbols.prioritySymbols.Highest} highest priority`,
100+
appendText: `${symbols.prioritySymbols.Highest} `,
101+
});
102102
genericSuggestions.push({
103103
displayText: `${symbols.prioritySymbols.Lowest} lowest priority`,
104104
appendText: `${symbols.prioritySymbols.Lowest} `,

tests/Suggestor/Suggestor.test.auto-complete_with__dataview__symbols_show_all_suggested_text.approved.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@
44
| due:: due date | due:: |
55
| start:: start date | start:: |
66
| scheduled:: scheduled date | scheduled:: |
7-
| priority:: highest highest priority | priority:: highest |
87
| priority:: high high priority | priority:: high |
98
| priority:: medium medium priority | priority:: medium |
109
| priority:: low low priority | priority:: low |
10+
| priority:: highest highest priority | priority:: highest |
1111
| priority:: lowest lowest priority | priority:: lowest |
1212
| repeat:: recurring (repeat) | repeat:: |
1313
| created:: created today (2022-07-11) | created:: 2022-07-11 |

tests/Suggestor/Suggestor.test.auto-complete_with__emoji__symbols_show_all_suggested_text.approved.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@
44
| 📅 due date | 📅 |
55
| 🛫 start date | 🛫 |
66
| ⏳ scheduled date ||
7-
| 🔺 highest priority | 🔺 |
87
| ⏫ high priority ||
98
| 🔼 medium priority | 🔼 |
109
| 🔽 low priority | 🔽 |
10+
| 🔺 highest priority | 🔺 |
1111
| ⏬ lowest priority ||
1212
| 🔁 recurring (repeat) | 🔁 |
1313
| ➕ created today (2022-07-11) | ➕ 2022-07-11 |

tests/Suggestor/Suggestor.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ describe.each([
4848
{ displayText: `${dueDateSymbol} due date`, appendText: `${dueDateSymbol} ` },
4949
{ displayText: `${startDateSymbol} start date`, appendText: `${startDateSymbol} ` },
5050
{ displayText: `${scheduledDateSymbol} scheduled date`, appendText: `${scheduledDateSymbol} ` },
51-
{ displayText: `${prioritySymbols.Highest} highest priority`, appendText: `${prioritySymbols.Highest} ` },
5251
{ displayText: `${prioritySymbols.High} high priority`, appendText: `${prioritySymbols.High} ` },
52+
{ displayText: `${prioritySymbols.Medium} medium priority`, appendText: `${prioritySymbols.Medium} ` },
5353
]);
5454
});
5555

0 commit comments

Comments
 (0)