Closed (fixed)
Project:
Panopoly
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Reporter:
Created:
19 Mar 2012 at 02:43 UTC
Updated:
23 Apr 2013 at 21:40 UTC
This patch modifies the height of the Ctools modals so our design looks right. Ideally this can be accomplished with $.extend() in the Ctools modal settings.
| Comment | File | Size | Author |
|---|---|---|---|
| ctools-modal-sizing.patch | 284 bytes | populist |
Comments
Comment #1
populist commentedThere is probably a better way to do this by overriding the entire settings array that Ctools sets up. Assigning this to me and marking as active.
Comment #2
populist commentedAnd there is, in fact, a better way! Here is the code I used to modify the default Chaos Tools modal:
Comment #4
sylus commentedSorry for reopening this.
I was wanting to apply my own custom style to CToolsModal but it seems I can't copy what is in comment #2 and put it in my own hook_init. If I commit out the one in panopoly_core then everything works and I don't recieve a javascript conflict.
Would it be possibly to wrap this CTools customization in a variable_get in case someone wants to use their own custom style overriding CToolsModal?
Comment #5
sylus commentedElevating priority as not sure how to work around this ^_^
Comment #6
populist commentedSounds reasonable. I moved the logic to Panopoly Magic as per #1899368: Allow Panopoly Magic to Work Without Panopoly Core, but this is now how the modal is called:
Comment #7
populist commentedCommitted to dev