Ticket #8531 (closed Task: wontfix)

Opened 2 years ago

Last modified 2 years ago

No   hack needed on tables

Reported by: fredck Owned by:
Priority: Normal Milestone:
Component: General Version:
Keywords: Cc:

Description

The   trick we use on empty tables to show them is not needed. It can be solved by adding the following CSS to the editing area:

table { empty-cells: show; }

Not tested though.

Source: Show empty cells in data tables with empty-cells: show.

Attachments

test3.html (657 bytes) - added by j.swiderski 2 years ago.

Change History

comment:1 Changed 2 years ago by j.swiderski

  • Status changed from new to confirmed

As discussed earlier with @fredck - this feature was suppose to be used for the WYSIWYG area.

I have tried to test the issue a little bit. Here are my findings:

The only code for filling table I have found, was the one from _source\plugins\table\dialogs\table.js:

if ( !CKEDITOR.env.ie )
	cell.append( makeElement( 'br' ) );

I haven't found any code that would fill the table with   (Well except when switching to source from wysiwyg).

If you apply the style provided by @fredck and remove the ‘if’ mentioned above, cells in non-IE browsers will be collapsed. They will get line-height if you put cursor in them (Firefox) or type something in side them (Chrome).


As I was saying before I didn’t find the code for filling the table in IE so I have prepared a simple test-case file – empty table with only one   in first cell.

Results: None of the browsers showed empty table cells with line heights. The were either collapsed or invisible (IE6, IE7). Only the cells with   had expected looks.


IMHO it will be hard to get rid off  . Anyway I'm confiming this ticket as it was understood:)

Version 1, edited 2 years ago by j.swiderski (previous) (next) (diff)

Changed 2 years ago by j.swiderski

comment:2 Changed 2 years ago by j.swiderski

  • Status changed from confirmed to closed
  • Resolution set to wontfix

I'm closing this as won't fix please reopen if I have missed something and there is a way to use it in all browsers.

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy