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

Add tokens for labels on activity edit page #5385

Merged
merged 4 commits into from
Feb 23, 2024
Merged

Add tokens for labels on activity edit page #5385

merged 4 commits into from
Feb 23, 2024

Conversation

jorg-vr
Copy link
Contributor

@jorg-vr jorg-vr commented Feb 23, 2024

This pull request reintroduces a token based editor for activity labels on the activity edit page.

The same label editor component is used as for user course labels, which I generalized.

The labels shown in the picker are all existing labels in the repository of the activity. (This seemed the most relevant scope)

Peek 2024-02-23 14-32

While working on this I also slightly improved the component. Small fixes include:

  • propper use of screen width
  • no unneeded padding in empty state
  • Already present labels are removed from dropdown
  • No auto select on typing full word (but do auto select on tab or enter)

Closes #5055

@jorg-vr jorg-vr added the bug Something isn't working label Feb 23, 2024
@jorg-vr jorg-vr self-assigned this Feb 23, 2024
@jorg-vr jorg-vr marked this pull request as ready for review February 23, 2024 13:38
@jorg-vr jorg-vr requested a review from a team as a code owner February 23, 2024 13:38
@jorg-vr jorg-vr requested review from bmesuere and niknetniko and removed request for a team February 23, 2024 13:38
@jorg-vr jorg-vr merged commit c876588 into main Feb 23, 2024
11 of 13 checks passed
@jorg-vr jorg-vr deleted the fix/label-edit branch February 23, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Label edit on activity edit page is broken
3 participants