-
Notifications
You must be signed in to change notification settings - Fork 55
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
Admin/MaterialList add icon #5537
base: devel
Are you sure you want to change the base?
Conversation
⛔ Feature branch deployment currently inactive.If the PR is still open, you can add the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could maybe add the icon on all places where we display the material list name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would want to discuss this in the core meeting before we merge this.
Did you discuss this yet? |
The discussion will also be about whether we will actively use the relation from material list to user from now on, and what implications this will have on collaborators entering and leaving and re-entering the camp, claiming invitations sent to other emails etc. |
Admin / Materiallist
Add icon
if it is a User-MaterialList, add UserAvatar
else add mdi-format-list-bulleted-square