Problem/Motivation

This is another small step in simplifying the dialog.
It does not require any special testing and backward compatibility because:

- jQuery will also respond to it
- There are no arguments in this event

I left the listeners for this event as is to show that nothing changes after this merge request

Issue fork drupal-3445033

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

finnsky created an issue. See original summary.

finnsky opened merge request !7898

finnsky’s picture

finnsky commented
Status: Active » Needs review
Issue tags: -JavaScript +JavaScript
finnsky’s picture

finnsky commented
Title: Replace dialogContentResize jQuery event » Replace dialogContentResize jQuery event with CustomEvent
Issue tags: -JavaScript +JavaScript
finnsky’s picture

finnsky commented
Issue tags: +Drupal 11 beta should-have
finnsky’s picture

finnsky commented
Issue tags: +beta target
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

So for this one I applied the MR locally.

Looking at the files I went to a view page (Content for this test)
While adding a new field in the popup I did a resize and everything seemed to behave as before

I enabled layout builder and enabled for Article content type
Adding a section I dragged the off canvas region without issue.

Believe this one is good.

  • nod_ committed ba3d4948 on 10.3.x 
    Issue #3445033 by finnsky, smustgrave: Replace dialogContentResize...

  • nod_ committed c699c850 on 10.4.x 
    Issue #3445033 by finnsky, smustgrave: Replace dialogContentResize...

  • nod_ committed 515bae7d on 11.0.x 
    Issue #3445033 by finnsky, smustgrave: Replace dialogContentResize...

  • nod_ committed 5686b0c3 on 11.x 
    Issue #3445033 by finnsky, smustgrave: Replace dialogContentResize...

nod_ closed merge request !7898

nod_’s picture

nod_
Primary language French
Location Lille
commented
Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 5686b0c and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.