Skip to content
Snippets Groups Projects

Installing drupal/edit_media_modal (2.0.0)

Merged Michael Lee requested to merge lee5151 into main

Module Addition(s)/Update(s)

Module(s) Added

  • Edit Media Modal - 2.0.0
    • Summary of Changes
      • Adds a button to Media in the CKEditor WYSIWYG. The button opens up a modal window to edit the media directly from the CKEditor instead of needing to navigate to the Edit media form.

Checklist

Check any that apply and include notes where appropriate.

  • Changes require an update.php
  • Changes require a configuration push
    • Needs to be installed after code push
  • Changes impact the user experience
  • Changes impact the administrative experience
    • Editors will now be able to more easily edit Media information through the CKEditor
  • Changes impact accessibility
    • Modal window and CKEditor functionality has some a11y limitations. We are not currently 100% compliant in admin a11y in general so we can address this as we shift focus there.
  • Changes impact security
  • Changes include updates to dependency packages
  • Changes result in code being deprecated
  • Relevant Pivotal Tracker tickets has been documented properly

Unique Deployment Procedure

If changes require additional work outside of the normal deployment process, include those details here

  1. Deploy code to all sites
  2. Install module on Clean Source
  3. Push configuration to all sites

Additional Notes

Requiring a module for this level of functionality is probably not the most ideal situation but as long as we are using a media within CKEditor it's probably a good solution. In the future, we should consider shifting further away from a homogenous body field in favor of modular fields with something like Layout Builder (something we've considered in the future anyway). This would be ideal because we could separate things like text and media into different paragraphs and do away with the need for this module. It's a future issue, but something I wanted to have down in writing.

Merge request reports

Approval is optional

Merged by Brian CaniniBrian Canini 8 months ago (Jul 31, 2024 2:57pm UTC)

Merge details

  • Changes merged into with 6110b558.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading