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

feat: Add readonly for skeleton #1104

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

ArneGudermann
Copy link
Contributor

resolves #1100

@ArneGudermann ArneGudermann added the Priority: Low This issue can be considered with enough idle time. label Mar 15, 2024
@ArneGudermann ArneGudermann added this to the ViUR-core v3.7 milestone Mar 15, 2024
@ArneGudermann ArneGudermann changed the title Add readonly for Skeleton feat: Add readonly for skeleton Mar 15, 2024
@sveneberth
Copy link
Member

I rather imagine a readOnly state at skeleton level here.

So on the one hand an attribute that can be set at skeleton level.
Then the value is inherited when rendering (but not set in the bone), so that you can deactivate the readOnly state again and know which bones were originally readonly.
And the readonly should then also prevent a toDB. And possibly this should also be mentioned when rendering in the renderer. (maybe this could be interesting later for cases where an edit is not possible but a view is).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low This issue can be considered with enough idle time.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

None yet

2 participants