-
Notifications
You must be signed in to change notification settings - Fork 14
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
N21 2089 placeholder element media shelf #3401
N21 2089 placeholder element media shelf #3401
Conversation
…ia-shelf' into N21-2089-placeholder-element-media-shelf
:data-element-id="element.id" | ||
:key="element.id" |
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.
Are you sure that "key" is not needed anymore. There was a bug where the DOM was desynchronized with the data structure of the list that the Sortable component uses. The key attribute prevented that elements would duplicate in the DOM.
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.
From Vue 3 onwards, the key attributes in on "v-if/else" branches are generated automatically: https://v3-migration.vuejs.org/breaking-changes/key-attribute#on-conditional-branches. So, the keys are actually still there.
…ia-shelf' into N21-2089-placeholder-element-media-shelf
Quality Gate passedIssues Measures |
Short Description
Links to Ticket and related Pull-Requests
Changes
Data-security
Deployment
New Repos, NPM packages or vendor scripts
Screenshots of UI changes
Checklist before merging