Changeset 6296


Ignore:
Timestamp:
01/06/2011 09:32:02 AM (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