Skip to content
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

Populate list of suggestions matching @mentions #6728

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

acelaya
Copy link
Contributor

@acelaya acelaya commented Dec 17, 2024

Depends on #6727

Make the list of suggestions displayed when typing @mentions include matching users from those who already annotated the document and belong to the active group.

The list is sorted by username and capped at 10 items.

Todo

  • Arrow key navigation while keeping textarea focused
  • Select on Enter press while keeping textarea focused
  • Select on click, then focus textarea
  • Style items

@acelaya acelaya force-pushed the at-mentions-suggestions branch 3 times, most recently from 38f04d8 to 7f8e3e9 Compare December 18, 2024 10:02
@acelaya acelaya force-pushed the at-mentions-popover branch 2 times, most recently from 62c5232 to 412f14c Compare December 18, 2024 10:26
@acelaya acelaya force-pushed the at-mentions-suggestions branch 2 times, most recently from 450cb7f to c5c81e1 Compare December 18, 2024 11:27
@acelaya acelaya force-pushed the at-mentions-popover branch 4 times, most recently from 07dc484 to 07ebc15 Compare December 19, 2024 08:44
Base automatically changed from at-mentions-popover to main December 19, 2024 08:46
@acelaya acelaya force-pushed the at-mentions-suggestions branch 6 times, most recently from bd9669d to 22ee031 Compare December 19, 2024 09:19
@acelaya acelaya force-pushed the at-mentions-suggestions branch from 22ee031 to 3397468 Compare December 20, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant