- Issue created by @berdir
- Merge request !85Attempt to select correct address and geolocation widget when there are multiple instances → (Open) created by berdir
- 🇨🇭Switzerland berdir Switzerland
I opened a merge request against 8.x-3.x for now since that's what we're using. I think the problem also exists in 4.x, the change doesn't apply but it doesn't seem to be doing anything that would improve this.
- 🇨🇭Switzerland berdir Switzerland
Found the problem, I had to use the full name as key to allow separate configurations for each.
- 🇩🇪Germany christianadamski Berlin, Germany
Could you in more detail describe the scenario? What would I have to set up to recreate this?
- 🇨🇭Switzerland berdir Switzerland
Create a paragraph type with a geolocation + address field and a paragraph field that allows multiple values.
Imagine a company node type with multiple locations, each location is a map with address + other infos, like opening hours or so. you could kind of build that with single field and multiple values each, but if you also want to
Note that you will also run into 🐛 Geolocation Field in combination with Paragraphs (Ajax/Modal) handling Active when trying to do that as html_head apparently can't be added through an ajax response, so if you don't load any geolocation fields initially, it won't work. (this scenario would work I guess if you set the paragraph widget to use such a location paragraph type by default.
- 🇩🇪Germany christianadamski Berlin, Germany
I'm at DrupalCamp Berlin. Spend whole day looking at this. Not simple.
- 🇩🇪Germany christianadamski Berlin, Germany
The parent form is still the node form. So I don't see an obvious way to clearly identify the matching address form element relative to the geolocation form.
Right now I think it will come done to CSS selector guessing :(
- Merge request !86Issue #3484824 - Paragraphs support in Widgets → (Merged) created by christianadamski
-
christianadamski →
committed 8e6d6ebd on 4.x
Issue #3484824 - Paragraphs support in Widgets
-
christianadamski →
committed 8e6d6ebd on 4.x
- 🇩🇪Germany christianadamski Berlin, Germany
v4 is fixed. You opened a MR for v3. I haven't tested it. Did you extensively? Can I merge blindly?
- 🇨🇭Switzerland berdir Switzerland
The client confirmed it worked for their use case, but I didn't test it that extensively, and specifically didn't test if it still works for any other use case.
Feel free to leave it at needs review for others to confirm or just only fix it for 4.x, your call.
- 🇩🇪Germany christianadamski Berlin, Germany
v4 I already merged (and released) separately.
Hmm. Ok, I'll see if I get around to testing v3.
I did not test thoroughly, but MR!85 applies for me in v8.x-3.14, and it fixes this issue in my project. Thanks!