Changeset 6296


Ignore:
Timestamp:
01/06/11 09:32:02 (4 years ago)
Author:
alfonsoml
Message:

#5522 old IE doesn't apply type=text for text fields

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r6295 r6296  
    7575                <li><a href="http://dev.ckeditor.com/ticket/5930">#5930</a> : [IE] Style definitions are no longer lowercased.</li> 
    7676                <li><a href="http://dev.ckeditor.com/ticket/5361">#5361</a> : Preview window's title should reflect the title tag in full page mode.</li> 
     77                <li><a href="http://dev.ckeditor.com/ticket/5522">#5522</a> : [IE] In versions &lt; 8 or compatibility mode, type="text" was missing in text fields.</li> 
    7778        </ul> 
    7879        <h3> 
  • CKEditor/trunk/_source/plugins/forms/plugin.js

    r6241 r6296  
    150150                                                if ( name == 'input' ) 
    151151                                                { 
    152                                                         var type = element.getAttribute( 'type' ); 
    153  
    154                                                         if ( type == 'text' || type == 'password' ) 
    155                                                                 return { textfield : CKEDITOR.TRISTATE_OFF }; 
    156  
    157                                                         if ( type == 'button' || type == 'submit' || type == 'reset' ) 
    158                                                                 return { button : CKEDITOR.TRISTATE_OFF }; 
    159  
    160                                                         if ( type == 'checkbox' ) 
    161                                                                 return { checkbox : CKEDITOR.TRISTATE_OFF }; 
    162  
    163                                                         if ( type == 'radio' ) 
    164                                                                 return { radio : CKEDITOR.TRISTATE_OFF }; 
    165  
    166                                                         if ( type == 'image' ) 
    167                                                                 return { imagebutton : CKEDITOR.TRISTATE_OFF }; 
     152                                                        switch( element.getAttribute( 'type' ) ) 
     153                                                        { 
     154                                                                case 'button' : 
     155                                                                case 'submit' : 
     156                                                                case 'reset' : 
     157                                                                        return { button : CKEDITOR.TRISTATE_OFF }; 
     158 
     159                                                                case 'checkbox' : 
     160                                                                        return { checkbox : CKEDITOR.TRISTATE_OFF }; 
     161 
     162                                                                case 'radio' : 
     163                                                                        return { radio : CKEDITOR.TRISTATE_OFF }; 
     164 
     165                                                                case 'image' : 
     166                                                                        return { imagebutton : CKEDITOR.TRISTATE_OFF }; 
     167 
     168                                                                default : 
     169                                                                        return { textfield : CKEDITOR.TRISTATE_OFF }; 
     170                                                        } 
    168171                                                } 
    169172 
     
    188191                                else if ( element.is( 'input' ) ) 
    189192                                { 
    190                                         var type = element.getAttribute( 'type' ); 
    191  
    192                                         switch ( type ) 
    193                                         { 
    194                                                 case 'text' : 
    195                                                 case 'password' : 
    196                                                         evt.data.dialog = 'textfield'; 
    197                                                         break; 
     193                                        switch ( element.getAttribute( 'type' ) ) 
     194                                        { 
    198195                                                case 'button' : 
    199196                                                case 'submit' : 
     
    209206                                                case 'image' : 
    210207                                                        evt.data.dialog = 'imagebutton'; 
     208                                                        break; 
     209                                                default : 
     210                                                        evt.data.dialog = 'textfield'; 
    211211                                                        break; 
    212212                                        } 
     
    232232                                                var attrs = input.attributes, 
    233233                                                        type = attrs.type; 
     234                                                // Old IEs don't provide type for Text inputs #5522 
     235                                                if ( !type ) 
     236                                                        attrs.type = 'text'; 
    234237                                                if ( type == 'checkbox' || type == 'radio' ) 
    235238                                                        attrs.value == 'on' && delete attrs.value; 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy