#171: Unable to set textfield char width to 20
----------------------------------------------+-----------------------------
  Reporter:  [EMAIL PROTECTED]  |       Owner:  martinkou           
      Type:  Bug                              |      Status:  assigned          
  
  Priority:  Normal                           |   Milestone:  FCKeditor 2.6     
  
 Component:  General                          |     Version:  FCKeditor 2.4     
  
Resolution:                                   |    Keywords:  SF Confirmed 
Review-
----------------------------------------------+-----------------------------
Changes (by alfonsoml):

  * keywords:  SF Confirmed Review? => SF Confirmed Review-

Comment:

 The problem that I see is that it does add even more preprocessing time to
 cover just a  very specific problem. And there are other issues that
 should be added if we are trying to protect all the IE problems. For
 example you have protected the textarea:cols, but I'm sure that there's a
 default value for rows that will make it also disappear.

 I remember that in another bug the problem was the margin properties of
 the body, in that case the values were something like 15 10 15 10 or the
 like

 If someone really cares about the dimensions then he will be using styles,
 and after all if the problem is present only when the property has its
 default value I don't see a big problem with it. I mean: not big enough as
 to add more time, there are people complaining that loading big HTML files
 in the editor is slow, every regexp that we add can make the problem even
 worse.

 Also, this is an IE only issue, so at the very least, those regexp aren't
 needed for the rest of browsers.

 I think that if this patch is added it should be applied only in the IE
 code.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/171#comment:9>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to