Changes between Version 1 and Version 2 of Ticket #8531, comment 1


Ignore:
Timestamp:
05/14/12 15:18:03 (2 years ago)
Author:
j.swiderski
Comment:

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. 
     1As discussed earlier with @fredck - this feature was supposed to be used for the WYSIWYG area. 
    22 
    33I have tried to test the issue a little bit. Here are my findings:[[BR]][[BR]] 
    44 
    5 The only code for filling table I have found, was the one from {{{_source\plugins\table\dialogs\table.js}}}: 
     5I’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}}}: 
    66{{{ 
    77if ( !CKEDITOR.env.ie ) 
    88        cell.append( makeElement( 'br' ) ); 
    99}}} 
    10 I haven't found any code that would fill the table with   (Well except when switching to source from wysiwyg).[[BR]] 
     10I 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---- 
     12Style {{{ 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---- 
     14As 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. 
    1115 
    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 &nbsp; had expected looks (the same result can be achieved with <br /> put inside table cell). 
    1417---- 
    1518 
    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 &nbsp; 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 &nbsp; or < br /> like we do. 
    1720 
    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 &nbsp; had expected looks. 
    19  
    20  
    21 ---- 
    22  
    23 IMHO it will be hard to get rid off &nbsp;.  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. 
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy