#2254: Malfunction in FCK.Selecte object
------------------------+---------------------------------------------------
  Reporter:  jenka1980  |       Owner:  martinkou        
      Type:  Bug        |      Status:  assigned         
  Priority:  Normal     |   Milestone:  FCKeditor 2.6.1  
 Component:  General    |     Version:  FCKeditor 2.5    
Resolution:             |    Keywords:  Confirmed Review-
------------------------+---------------------------------------------------
Changes (by martinkou):

  * keywords:  Confirmed Review? => Confirmed Review-
  * owner:  w.olchawa => martinkou
  * status:  new => assigned

Comment:

 Hmm... the patch has a number of problems:

  1. toLocalUpperCase() is not defined, which gives out JavaScript errors.
  1. Even converting the toLocalUpperCase() to toUpperCase(), the patch is
 still wrong because it still can't check against DOM nodes with lowercase
 tag names.

 We actually have a shorthand string function defined in FCKeditor's code
 for this kind of case insensitive string matching - String::IEquals(). You
 can find it in $FCKeditor/editor/_source/fckjscoreextensions.js, it's a
 very useful function. :)

-- 
Ticket URL: <https://dev.fckeditor.net/ticket/2254#comment:4>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to