Opened 5 years ago

Closed 5 years ago

#8531 closed Task (wontfix)

No   hack needed on tables

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


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 (1)

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

Download all attachments as: .zip

Change History (3)

comment:1 Changed 5 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 ( ! )
	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 5 years ago by j.swiderski (previous) (next) (diff)

Changed 5 years ago by j.swiderski

comment:2 Changed 5 years ago by j.swiderski

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

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 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy