Skip to content

fix: Enable 'High priority' to be selected easily in Auto Suggest #1996

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

Merged
merged 1 commit into from
May 30, 2023

Conversation

claremacrae
Copy link
Collaborator

@claremacrae claremacrae commented May 30, 2023

Description

Via auto-suggest I keep ending up with tasks of Highest priority accidentally. See #1995 for a write-up.

This PR reverts to the original order of priority items in the auto-suggest menu.

The order of priorities in auto suggestion is now:

  1. high
  2. medium
  3. low
  4. highest (new)
  5. lowest (new)

Benefits:

  • 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.

Motivation and Context

Fixes #1995

How has this been tested?

  • Via the unit tests
  • In Obsidian (both with the default number of suggests (6) and the maximum (12)

Screenshots (if appropriate)

With 6 suggestions:

image

With 12 suggestions:

image

Types of changes

Changes visible to users:

  • Bug fix (prefix: fix - non-breaking change which fixes an issue)
  • Documentation (prefix: docs - improvements to any documentation content for users)

Internal changes:

  • Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

Terms

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.
@claremacrae claremacrae added the scope: auto-suggest The pop-up menu that helps with editing tasks directly in markdown notes label May 30, 2023
@claremacrae claremacrae merged commit d63f228 into main May 30, 2023
@claremacrae claremacrae deleted the auto-suggest-issue-1995 branch May 30, 2023 21:33
Copy link
Collaborator Author

@claremacrae claremacrae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes #1943

This commit message should have linked to #1995...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: auto-suggest The pop-up menu that helps with editing tasks directly in markdown notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Via auto-suggest I keep ending up with tasks of Highest priority accidentally
1 participant