Ticket #2215: 2215.4.patch

_whatsnew.html

@@ -42 +42 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2150">#2150</a>] The searching
                         speed of the Find/Replace dialog has been vastly improved.</li>
                 <li>New language file for <strong>Gujarati</strong> (by Nilam Doctor).</li>
+                <li>A new TabIndex property has been added to the JavaScript integration files.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2215">#2215</a>] Following
+                        the above new feature, the ReplaceTextarea method will now copy the textarea.tabIndex
+                        value if available.</li>
 
                 
         </ul>

fckeditor.js

@@ -88 +88 @@
         {
                 var sWidth  = this.Width.toString().indexOf('%')  > 0 ? this.Width  : this.Width  + 'px' ;
                 var sHeight = this.Height.toString().indexOf('%') > 0 ? this.Height : this.Height + 'px' ;
-                sHtml += '<textarea name="' + this.InstanceName + '" rows="4" cols="40" style="width:' + sWidth + ';height:' + sHeight + '">' + this._HTMLEncode( this.Value ) + '<\/textarea>' ;
+
+                sHtml += '<textarea name="' + this.InstanceName +
+                        '" rows="4" cols="40" style="width:' + sWidth +
+                        ';height:' + sHeight ;
+
+                if ( this.TabIndex )
+                        sHtml += '" tabindex="' + this.TabIndex ;
+
+                sHtml += '">' +
+                        this._HTMLEncode( this.Value ) +
+                        '<\/textarea>' ;
         }
 
         return sHtml ;
@@ -116 +126 @@
                 }
 
                 oTextarea.style.display = 'none' ;
+
+                if ( oTextarea.tabIndex )
+                        this.TabIndex = oTextarea.tabIndex ;
+
                 this._InsertHtmlBefore( this._GetConfigHtml(), oTextarea ) ;
                 this._InsertHtmlBefore( this._GetIFrameHtml(), oTextarea ) ;
         }
@@ -158 +172 @@
         catch (e) { /* Ignore it. Much probably we are inside a FRAME where the "top" is in another domain (security error). */ }
 
         var sLink = this.BasePath + 'editor/' + sFile + '?InstanceName=' + encodeURIComponent( this.InstanceName ) ;
-        if (this.ToolbarSet) sLink += '&amp;Toolbar=' + this.ToolbarSet ;
+        if (this.ToolbarSet)
+                sLink += '&amp;Toolbar=' + this.ToolbarSet ;
 
-        return '<iframe id="' + this.InstanceName + '___Frame" src="' + sLink + '" width="' + this.Width + '" height="' + this.Height + '" frameborder="0" scrolling="no"></iframe>' ;
+        html = '<iframe id="' + this.InstanceName +
+                '___Frame" src="' + sLink +
+                '" width="' + this.Width +
+                '" height="' + this.Height ;
+
+        if ( this.TabIndex )
+                html += '" tabindex="' + this.TabIndex ;
+
+        html += '" frameborder="0" scrolling="no"></iframe>' ;
+
+        return html ;
 }
 
 FCKeditor.prototype._IsCompatibleBrowser = function()