We've themed CTools Modal and worked on a new look for Panels IPE modal. Help us test it.

Radix Panels IPE modal

CommentFileSizeAuthor
#8 2190325-radix-modal-8.patch624 bytesguillaumev
#7 screenshot-2190325.png141.08 KBguillaumev
#4 radix-responsive_ctools_categories.patch959 bytesbarraponto
radix-ctools-modal-panels-ipe.png245.21 KBshadcn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

shadcn’s picture

shadcn CreditAttribution: shadcn commented
Issue summary: View changes
barraponto’s picture

barraponto CreditAttribution: barraponto commented

Does this rely on CTools 1.4 (pre-release)?

shadcn’s picture

shadcn CreditAttribution: shadcn commented

Using CTools 7.x-1.3 from Panopoly dev branch.

barraponto’s picture

barraponto CreditAttribution: barraponto commented
FileSize
radix-responsive_ctools_categories.patch959 bytes

Adding some responsiveness to the columns for smaller devices.

shadcn’s picture

shadcn CreditAttribution: shadcn commented

Committed. Thanks.

barraponto’s picture

barraponto CreditAttribution: barraponto commented
Status: Needs work » Active

See also #2223691: Use .modal-open to prevent scrolling outside the modal. Leaving as Active since there's no patch to work or review.

guillaumev’s picture

guillaumev CreditAttribution: guillaumev commented
FileSize
screenshot-2190325.png141.08 KB

Hi,

I'm having an issue with radix 3.x-dev and modals. When using modals with lots of content, the modal is not resized properly and the form simply ends up not working. See the attached screenshot...

guillaumev’s picture

guillaumev CreditAttribution: guillaumev commented
Status: Active » Needs review
FileSize
2190325-radix-modal-8.patch624 bytes

Not sure this patch is the cleanest way to fix it, but it fixes it for me...

migueltrindade’s picture

migueltrindade CreditAttribution: migueltrindade commented

When you close the modal using the "Esc" button it doesn't remove "modal-open" class from body.

chriz001’s picture

chriz001 CreditAttribution: chriz001 commented

Had the same problem with the 'esc' button, this should fix it. I can make a patch if needed.

  $(document).on('CToolsDetachBehaviors', function(event, modalContent) {
    $('body').removeClass('modal-open');
  });
shadcn’s picture

shadcn CreditAttribution: shadcn commented
Status: Needs review » Fixed

Marking this issue as fixed. Feel free to reopen if you have any question. Thanks.

Status: Fixed » Closed (fixed)

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