-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
[BITV] 9.1.1.1a/3.7 - "Details" section / The combo box for selecting "Tags" - The button for deleting a tag does not have an alternative text. (2) #36718
Comments
Functionality additions to NcSelectTags being done in nextcloud-libraries/nextcloud-vue#3946 |
Options considered thus far are using NcSelectTags in https://github.com/nextcloud/server and passing prop overrides for systemtags functionality OR adding systemtags functionality to NcSelectTags itself as it already contains some systemtags specific code Both of these options are infeasible in one way or another due to the discrepancy between internal and external state incurred by adding select, create, and delete functionality For this reason as well as to maintain compatibility and to have an intuitive implementation we will need to make changes in https://github.com/nextcloud/nextcloud-vue and https://github.com/nextcloud/server An opt-in prop will need to be added to override the custom internal state handling of NcSelectTags and all new systemtags specific logic will need to be done in the systemtags app |
Server changes done in #37065 |
[WIP] Edit system tags nextcloud-libraries/nextcloud-vue#3946Details
https://report.bitvtest.de/default-en/d63601ac-cb34-4645-8256-66bec78964a0.html#checkpoint-cd239865a5-v3-n7
The text was updated successfully, but these errors were encountered: