Changeset 4923


Ignore:
Timestamp:
01/14/2010 09:49:40 PM (5 years ago)
Author:
alfonsoml
Message:

#4773 make the fileBrowser plugin respect onClick handlers

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r4921 r4923  
    5151                <li><a href="http://dev.fckeditor.net/ticket/4863">#4863</a> : Fixing iframedialog's height doesn't stretch to 100%</li> 
    5252                <li><a href="http://dev.fckeditor.net/ticket/4964">#4964</a> : The BACKSPACE key positioning was not correct in some cases with Firefox.</li> 
     53                <li><a href="http://dev.fckeditor.net/ticket/4773">#4773</a> : The fileBrowser plugin overwrote any onClick function for a fileButton element.</li> 
    5354        </ul> 
    5455        <h3> 
  • CKEditor/trunk/_source/plugins/filebrowser/plugin.js

    r4918 r4923  
    279279                                if ( url ) 
    280280                                { 
    281                                         element.onClick = uploadFile; 
     281                                        var onClick = element.onClick; 
     282                                        element.onClick = function( evt ) 
     283                                        { 
     284                                                // "element" here means the definition object, so we need to find the correct 
     285                                                // button to scope the event call 
     286                                                var sender = evt.sender; 
     287                                                if ( onClick && onClick.call( sender, evt ) === false ) 
     288                                                        return false; 
     289 
     290                                                return uploadFile.call( sender, evt ); 
     291                                        }; 
     292 
    282293                                        element.filebrowser.url = url; 
    283294                                        element.hidden = false; 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy