Changes between Version 1 and Version 2 of Ticket #8531, comment 1
- Timestamp:
- May 14, 2012, 1:18:03 PM (13 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #8531, comment 1
v1 v2 1 As discussed earlier with @fredck - this feature was suppose to be used for the WYSIWYG area.1 As discussed earlier with @fredck - this feature was supposed to be used for the WYSIWYG area. 2 2 3 3 I have tried to test the issue a little bit. Here are my findings:[[BR]][[BR]] 4 4 5 The only code for filling table I have found,was the one from {{{_source\plugins\table\dialogs\table.js}}}:5 I’m not sure what @fredck meant by trick as the only code for filling table I have found was the one from {{{_source\plugins\table\dialogs\table.js}}}: 6 6 {{{ 7 7 if ( !CKEDITOR.env.ie ) 8 8 cell.append( makeElement( 'br' ) ); 9 9 }}} 10 I haven't found any code that would fill the table with (Well except when switching to source from wysiwyg).[[BR]] 10 I haven't found any code that would fill the table with (Well except when switching to source from wysiwyg - but it has nothing to do presentation of the table).[[BR]] 11 ---- 12 Style {{{ empty-cells: show;}}}is used to render background and borders of empty table cells. According to the article it should work in older versions of IE but **in my tests it didn’t work in IE6 and IE7** (empty table cells were not visible). Furthermore if you apply style provided by @fredck and remove the ‘if’ mentioned above, cells in non-IE browsers will be collapsed. The border and background will be visible but there will be no height. Cells will get line-height if you put cursor in them (Firefox) or type something inside them (Chrome). 13 ---- 14 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. 11 15 12 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). 13 16 **Results:** Borders and background were shown but none of the browsers showed empty table cells with line heights. They were either collapsed (IE8, IE9) or invisible (IE6, IE7). Only the cells with had expected looks (the same result can be achieved with <br /> put inside table cell). 14 17 ---- 15 18 16 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.19 **To summarize**. This style does not work in IE6 and IE7. Furthermore empty cells are collapsed so it looks to me like the only way to handle this is to use or < br /> like we do. 17 20 18 **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. 19 20 21 ---- 22 23 IMHO it will be hard to get rid off . Anyway I'm confiming this ticket as it was understood:) 21 **NOTE:** You can always add height to cell but still its border will not be visible in IE6-7.
