- Issue created by @max.valetov
- Status changed to Needs work
6 months ago 7:58am 4 June 2024 - 🇳🇱Netherlands johnv
I have some questions about your patch.
- the patch is not against latest version, with for example part of 📌 Refactor the JavaScript for the widget to use the once library Needs review in it. That is not a problem however to evaluate your patch. But it does already introduce some more focussing.
- you have a problem in a specific paragraphs constellation. I presume you have no problem in a less complex layout? (there were some issues with paragraphs before)
- in the first 2 parts, I guess the addClass/removeClass is the part that truely fixes your problem?
- in the first 2 parts you change the UX: removing the cleared row and adding the 'Add' link of the previous row. I guess that is not part of your problem, but an additional feature? Indeed , current behaviour is more like 'clear', then 'remove'. But I feel removing the clicked-upon row is confusing.
- in the last 2 parts, why do you switch from 'id' to 'data-drupal-selector'? Is that best practice? If so, I can change it, but it gives no technical/functional improvement, does it?Additionally, please check the introduction of once in #3449164-11: Refactor the JavaScript for the widget to use the once library → , if it will not break your use case.
I need some confirmation on the proposed change there. - 🇳🇱Netherlands johnv
Please find the remainder of your original patch, for the hide and once features.
- 🇧🇪Belgium andreasderijcke Antwerpen / Gent
This does occur in other nested situations too, when using field_group (for example) to put the field in:
- details element
- tabs
- ...Tested on the last DEV commit at moment of writing.
Will try to update the patch (or have a colleague do it), as it doesn't apply on the last DEV commit.
- 🇳🇱Netherlands johnv
Thanks @andreasderijcke.
Please note that I just released a new version 1.18. - Assigned to andreasderijcke
- Issue was unassigned.
- 🇧🇪Belgium andreasderijcke Antwerpen / Gent
Getting 📌 Refactor the JavaScript for the widget to use the once library Needs review fixed first might make fixing this easier, if only to avoid a merge hell.
- Status changed to Needs review
2 months ago 7:39pm 20 September 2024 - 🇧🇪Belgium andreasderijcke Antwerpen / Gent
📌 Refactor the JavaScript for the widget to use the once library Needs review fixes this problem as far as I could reproduce and test it.
@max.valetov Can you check if this is the case for you too?
- 🇳🇱Netherlands johnv
Thanks, the remainder is indeed fixed in 📌 Refactor the JavaScript for the widget to use the once library Needs review
Automatically closed - issue fixed for 2 weeks with no activity.