Add optional initial_query to suggester

[mnaoumov]

Fixes #555

[Zachatoo]

Could you also update the docs as part of this PR?

[AB1908]

Can someone remind me how this is different from placeholder? Also, for conformity, should we rename it to default_value?

[tektrip-biggles]

Can someone remind me how this is different from placeholder? Also, for conformity, should we rename it to default_value?

My understanding is that placeholder simply appears in the prompt box as an aide memoire, whereas default_value would be a valid option from the list presented to the user. E.g. I want to be prompted for task priority so I might use:

<% tp.system.suggester(
  (item) => item, 
  ["1: Highest", "2: High", "3: Medium", "4: Low", "6: Lowest"], 
  false, 
  "Priority",
   5, 
  "3: Medium") 
%>

This way if I press enter as soon as the prompt appears, I'll select "3: Medium" as priority instead of "1: Highest" as currently. This saves two keypresses in the average case and four keypresses in the worst case, which can add up quickly when, say, adding a ton of task notes.

Don't suppose there's any chance this will make its way into main any time soon btw?

[Zachatoo]

Cleaning up stale PRs.

[mnaoumov]

@Zachatoo Can I do something for it to be reopen and merged?

[Zachatoo]

1. Can we rename it to default_value?
2. Can you also add this to tp.system.multi_suggester?
3. Can you update the docs (documentation.toml)?

If you can do at least 1 and 2 I'll accept this. 3 would be nice but not required, I'll do it if you don't.

[mnaoumov]

Sure, I will do all 3

[mnaoumov]

@Zachatoo ready to review

[Zachatoo]

This doesn't appear to be working as expected, the user still has to select the option for it to work for the multi-suggester.

IMO we should pass an array of options in the template as the default values and have them pre-selected, instead of just setting the input value.

[mnaoumov]

@Zachatoo ready for review

[Zachatoo]

Great, thanks!