Ticket #3492: 3492.patch

File 3492.patch, 5.4 KB (added by Tobiasz Cudnik, 12 years ago)

Testes in FF3, IE8, IE6

  • _source/plugins/smiley/dialogs/smiley.js

     
    99                images = config.smiley_images,
    1010                columns = config.smiley_columns,
    1111                i;
     12       
     13        // Simulate "this" of a dialog for non-dialog events.
     14        var dialog;
     15        var onClick = function( evt )
     16        {
     17                var target = evt.data.getTarget(),
     18                        targetName = target.getName();
     19
     20                if ( targetName == 'td' )
     21                        target = target.getChild( [ 0, 0 ] );
     22                else if ( targetName == 'a' )
     23                        target = target.getChild( 0 );
     24                else if ( targetName != 'img' )
     25                        return;
     26
     27                var src = target.getAttribute( 'cke_src' ),
     28                        title = target.getAttribute( 'title' );
     29
     30                var img = editor.document.createElement( 'img',
     31                        {
     32                                attributes :
     33                                {
     34                                        src : src,
     35                                        _cke_saved_src : src,
     36                                        title : title,
     37                                        alt : title
     38                                }
     39                        });
     40
     41                editor.insertElement( img );
     42
     43                dialog.hide();
     44        }
     45               
     46        var onKeydown = CKEDITOR.tools.addFunction( function( ev, element )
     47        {
     48                ev = new CKEDITOR.dom.event( ev );
     49                element = new CKEDITOR.dom.element( element );
     50                var relative, nodeToMove;
     51
     52                var keystroke = ev.getKeystroke();
     53                switch ( keystroke )
     54                {
     55                        case 39 : // RIGHT-ARROW
     56                                // relative is TD
     57                                if ( ( relative = element.getParent().getNext() ) )
     58                                {
     59                                        nodeToMove = relative.getChild( 0 );
     60                                        nodeToMove.focus();
     61                                }
     62                                break;
     63                        case 37 : // LEFT-ARROW
     64                                // relative is TD
     65                                if ( ( relative = element.getParent().getPrevious() ) )
     66                                {
     67                                        nodeToMove = relative.getChild( 0 );
     68                                        nodeToMove.focus();
     69                                }
     70                                break;
     71                        case 38 : // UP-ARROW
     72                                // relative is TR
     73                                if ( ( relative = element.getParent().getParent().getPrevious() ) )
     74                                {
     75                                        nodeToMove = relative.getChild( [element.getParent().getIndex(), 0] );
     76                                        nodeToMove.focus();
     77                                }
     78                                break;
     79                        case 40 :       // ARROW-DOWN
     80                                // relative is TR
     81                                if ( ( relative = element.getParent().getParent().getNext() ) )
     82                                {
     83                                        nodeToMove = relative.getChild( [element.getParent().getIndex(), 0] );
     84                                        if ( nodeToMove )
     85                                                nodeToMove.focus();
     86                                }
     87                                break;
     88                        case 13 :       // ENTER
     89                        case 32 :       // SPACE
     90                                onClick( { data: ev } );
     91                                break;
     92                        default :
     93                                // Do not stop not handled events.
     94                                return;
     95                }
     96
     97                // Avoid subsequent focus grab on editor document.
     98                ev.preventDefault();
     99        });
    12100
    13101        // Build the HTML for the smiley images table.
    14102        var html =
     
    25113
    26114                html.push(
    27115                        '<td class="dark_background hand centered" style="vertical-align: middle;">' +
    28                                 '<img border="0" class="hand" title="', config.smiley_descriptions[i], '"' +
    29                                         ' cke_src="', CKEDITOR.tools.htmlEncode( config.smiley_path + images[ i ] ), '"',
    30                                         ' src="', CKEDITOR.tools.htmlEncode( config.smiley_path + images[ i ] ), '"',
    31                                         // IE BUG: Below is a workaround to an IE image loading bug to ensure the image sizes are correct.
    32                                         ( CKEDITOR.env.ie ? ' onload="this.setAttribute(\'width\', 2); this.removeAttribute(\'width\');" ' : '' ),
    33                                 '>' +
     116                                '<a class="cke_smile" tabindex="-1"  onkeydown="CKEDITOR.tools.callFunction( ', onKeydown, ', event, this );">',
     117                                        '<img class="hand" title="', config.smiley_descriptions[i], '"' +
     118                                                ' cke_src="', CKEDITOR.tools.htmlEncode( config.smiley_path + images[ i ] ), '"',
     119                                                ' src="', CKEDITOR.tools.htmlEncode( config.smiley_path + images[ i ] ), '"',
     120                                                // IE BUG: Below is a workaround to an IE image loading bug to ensure the image sizes are correct.
     121                                                ( CKEDITOR.env.ie ? ' onload="this.setAttribute(\'width\', 2); this.removeAttribute(\'width\');" ' : '' ),
     122                                        '>' +
     123                                '</a>',
    34124                        '</td>' );
    35125
    36126                if ( i % columns == columns - 1 )
     
    50140        {
    51141                type : 'html',
    52142                html : html.join( '' ),
    53                 onClick : function( evt )
     143                onShow : function( event )
    54144                {
    55                         var target = evt.data.getTarget(),
    56                                 targetName = target.getName();
    57 
    58                         if ( targetName == 'td' )
    59                                 target = target.getChild(0);
    60                         else if ( targetName != 'img' )
    61                                 return;
    62 
    63                         var src = target.getAttribute( 'cke_src' ),
    64                                 title = target.getAttribute( 'title' );
    65 
    66                         var img = editor.document.createElement( 'img',
    67                                 {
    68                                         attributes :
    69                                         {
    70                                                 src : src,
    71                                                 _cke_saved_src : src,
    72                                                 title : title,
    73                                                 alt : title
    74                                         }
    75                                 });
    76 
    77                         editor.insertElement( img );
    78 
    79                         this.getDialog().hide();
     145                        // Momorize dialog reference inside scope of this file.
     146                        dialog = event.sender;
     147                        var firstSmile = event.sender.parts.contents.getChild( [0, 0, 0, 0, 0, 0, 0, 0, 0, 0] );
     148                        firstSmile.focus();
     149                        event.sender._.hasFocus = true;
    80150                },
     151                onClick : onClick,
    81152                style : 'width: 100%; height: 100%; border-collapse: separate;'
    82153        };
    83154
  • _source/skins/v2/dialog.css

     
    591591{
    592592        color: #a0a0a0;
    593593}
     594
     595.cke_skin_v2 a.cke_smile img
     596{
     597        /* IE6 does not support transparent borders */
     598        border: 2px solid #eaead1;
     599}
     600
     601.cke_skin_v2 a.cke_smile:focus img,
     602.cke_skin_v2 a.cke_smile:active img,
     603.cke_skin_v2 a.cke_smile:hover img
     604{
     605        border-color: #C7C78F;
     606}
     607 No newline at end of file
© 2003 – 2021 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy