- Issue created by @mandclu
- 🇨🇦Canada mandclu
NOTE: As a stopgap, importing Popper from an external source can also work, as was done in this Webform commit.
- First commit to issue fork.
- 🇮🇳India rajeshreeputra Pune
There is another error while using Acquia DAM with Site Studio.
unexpected alert open: {Alert text : An error occurred while attempting to process /media-library?source=core&media_library_opener_id=media_library.opener.cohesion&media_library_allowed_types%5B0%5D=image&media_library_selected_type=image&media_library_remaining=1&hash=3QAwKMPIPLTV_agyGNBNd4K3KGsqfUVHZCAAk4JmawQ&_wrapper_format=drupal_ajax&complete_allowed_list%5B0%5D=image&complete_allowed_list%5B1%5D=acquia_dam_image_asset&ajax_form=1&views_display_id=widget&_wrapper_format=drupal_ajax: ajax.$form.ajaxSubmit is not a function}
- Merge request !9#3349130 – Provide Popper.js as part of the module → (Open) created by rajeshreeputra
- 🇮🇳India rajeshreeputra Pune
Meanwhile the migration work getting completed, created patch to provide PopperJS as part of module. With MR !9 this is working.
@mandclu could you please apply the patch and verify if it works for you. - Assigned to mglaman
- Status changed to Needs review
over 1 year ago 3:00pm 13 April 2023 - Status changed to Needs work
over 1 year ago 3:05pm 13 April 2023 - 🇺🇸United States mglaman WI, USA
I don't really want to embed Popper.js and fork the code. I'd rather find ways to avoid calling Popper if it is not set, removing the tooltips.
- Merge request !10#3349130 – Javascript error in Drupal 10: migrate Popper.js to FloatingUI → (Open) created by mglaman
- Status changed to Needs review
over 1 year ago 3:33pm 13 April 2023 - 🇺🇸United States mglaman WI, USA
We only use Popper.js for the media overview page. Otherwise the CKE5 tooltips are used. This simply removes the tooltip on D10 for the media overview page.
- Status changed to Needs work
over 1 year ago 5:45pm 13 April 2023 - 🇺🇸United States mglaman WI, USA
I need to sort this out, https://git.drupalcode.org/project/acquia_dam/-/merge_requests/10/diffs?... fixes D10 failures. But now there a bunch of random failures on D9.
- 🇺🇸United States mglaman WI, USA
Okay, the scope of the MR is back to just fixing JavaScript code for Popper.
- Issue was unassigned.
- Status changed to Fixed
over 1 year ago 2:04pm 1 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.