Opened 12 years ago

Last modified 8 years ago

#10021 confirmed Bug

Table plugin uses attributes deprecated in HTML5 — at Version 7

Reported by: Danil Owned by:
Priority: Nice to have (we want to work on it) Milestone:
Component: Core : Tables Version:
Keywords: Cc:

Description (last modified by Jakub Ś)

When I insert new table, without any settings i get border="1" cellpadding="1" cellspacing="1" style="width: 500px;" attributes. These are unexpected and non-valid ones.

Edit:
When I insert new table it uses attributes that are deprecated in HTML5 cellpadding,cellspacing,align,summary,rules,frame,bgcolor attributes. Please see: http://www.w3schools.com/tags/tag_table.asp

Change History (6)

comment:2 Changed 12 years ago by Danil

How can I get rid of these deprecated attributes? I use stylesheet file to style plain tables (I guess most modern sites use it). Is any settings to tweak table plugin? It should at least use css to have any effect.

comment:3 Changed 12 years ago by Jakub Ś

Description: modified (diff)
Summary: Insert table plugin adds unexpected attributesTable plugin uses attributes deprecated in HTML5

comment:4 Changed 12 years ago by Jakub Ś

Description: modified (diff)

comment:5 Changed 12 years ago by Jakub Ś

Component: Core : EditableCore : Tables
Description: modified (diff)
Status: newconfirmed

@danya_postfactum I have modified your description as you are right - CKEditor uses some old and deprecated attributes that should be replaced by styles in HTML5.

  1. When creating table one can always delete these values in table dialog (manually set values to zero or delete them)
  2. These default values are set in dialog plugin and the only way to change them is by setting them to empty values:
    CKEDITOR.on( 'dialogDefinition', function( ev )	{				
    				var dialogName = ev.data.name;
    				var dialogDefinition = ev.data.definition;						
    				if ( dialogName == 'table' ){
    					var infoTab = dialogDefinition.getContents( 'info' );
    					infoTab.get( 'txtBorder' ).default = '';
    					infoTab.get( 'txtWidth' ).default = '';
    					infoTab.get( 'txtCellSpace' ).default = '';
    					infoTab.get( 'txtCellPad' ).default = '';
    				}
    			}	);
    

comment:6 Changed 12 years ago by Danil

Thanks, this is more correct info. But I still think default width should be null.

comment:7 Changed 12 years ago by Jakub Ś

Description: modified (diff)
  1. These are not used by editor: rules,frame,bgcolor
  2. This can probably be dropped: summary
  3. This can perhaps be replaced by float in style attribute: - align
  4. There is a problem with: cellpadding and cellspacing

We still support IE7 and Quirks mode which makes this request very nice but impossible to implement at the given moment at least for these two attributes

Links: http://www.quackit.com/css/css_cellpadding.cfm
http://www.quackit.com/css/css_cellspacing.cfm
http://stackoverflow.com/questions/339923/how-to-set-cellpadding-and-cellspacing-in-css
http://www.w3schools.com/cssref/pr_tab_border-spacing.asp

Last edited 12 years ago by Jakub Ś (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy