Permalink
Browse files

MINOR Disable ui-titlebar styling for link/image forms, as it breaks …

…the "insert" button interactions (to be replaced with a new modal dialog implementation soon anyway)
  • Loading branch information...
1 parent f547da2 commit 842c38fbc41d76748626ef6cd443f0bf9c6a388d @chillu chillu committed Apr 5, 2011
Showing with 18 additions and 16 deletions.
  1. +18 −16 javascript/CMSMain.js
View
34 javascript/CMSMain.js
@@ -9,22 +9,24 @@
*
* All forms in the right content panel should have closeable jQuery UI style titles.
*/
- $('#contentPanel form').entwine({
- // Constructor: onmatch
- onmatch: function() {
- // Style as title bar
- this.find(':header:first').titlebar({
- closeButton:true
- });
- // The close button should close the east panel of the layout
- this.find(':header:first .ui-dialog-titlebar-close').bind('click', function(e) {
- $('body.CMSMain').entwine('ss').getMainLayout().close('east');
- return false;
- });
-
- this._super();
- }
- });
+ // TODO Change to modal dialog, jQuery UI update somehow messed up the button positioning,
+ // they're no longer clickable
+ // $('#contentPanel form').entwine({
+ // // Constructor: onmatch
+ // onmatch: function() {
+ // // Style as title bar
+ // this.find(':header:first').titlebar({
+ // closeButton:true
+ // });
+ // // The close button should close the east panel of the layout
+ // this.find(':header:first .ui-dialog-titlebar-close').bind('click', function(e) {
+ // $('body.CMSMain').entwine('ss').getMainLayout().close('east');
+ // return false;
+ // });
+ //
+ // this._super();
+ // }
+ // });
/**
* Class: #Form_SearchTreeForm

0 comments on commit 842c38f

Please sign in to comment.