Styles combo drop down list not refreshed

[Gilles van den Hoven]

Situation:

• CKEditor 3.4.2
  
• Custom styleset loaded
  
• removeFormatTags added to include (e.g. remove) block elements
  

Following configuration was used:

<textarea name="editor1" rows="8" cols="60">&lt;h1&gt;Initial value&lt;/h1&gt;</textarea>
<script type="text/javascript">//<![CDATA[
CKEDITOR.replace('editor1', {
		"language": "en",
		"skin": "v2",
		"width": "100%",
		"height": "500",
		"resize_enabled": false,
		"baseHref": "http:\/\/127.0.0.1\/",
		"forcePasteAsPlainText": true, 
		"removePlugins": "elementspath",
		"removeFormatTags": "b,big,code,del,dfn,em,font,i,ins,kbd,q,samp,small,span,strike,strong,sub,sup,tt,u,var,h1,h2,h3,h4,h5,h6,hr",
		"toolbarCanCollapse": false,
		"stylesSet": "default:http:\/\/127.0.0.1\/\/rewrite.php\/cms\/ckeditor\/styles"
	});
//]]></script>

The loaded styleset:

CKEDITOR.stylesSet.add('default', [ { name: 'CMS - Accordeon titel element', element: 'div', attributes: { 'class': 'wuaTitle' } }, { name: 'Koptekst 1', element: 'h1' }, { name: 'Koptekst 2', element: 'h2' }, { name: 'Koptekst 3', element: 'h3' }, { name: 'Koptekst 4', element: 'h4' }, { name: 'Koptekst 5', element: 'h5' }, { name: 'Koptekst 6', element: 'h6' } ]);

BUG description:

• Select the entire tag
  
• Click on the removeFormat button
  
• This removes the H1, but leaves the style marked in the stylescombo.
  
• So the styles combo did not get refreshed/updated in FF 3.6.12.
  

Does NOT happen (e.g. behaviour is correct) when the H1 is wrapped in a container as shown below:

<textarea name="editor1" rows="8" cols="60">&lt;div&gt;&lt;h1&gt;Initial value&lt;/h1&gt;&lt;/div&gt;</textarea>

Cause & sollution:
After searching and diggin into the code for a huge ammount of time..
It seems that after clicking the removeFormat button, the "selectionChange", attached in the stylescombo (onRender) is not fired.
Getting this event to fire (duplicating code from _source/plugins/tabletools/dialogs.js:525) was unsuccesfull.
I managed to get it working by doing the following 2 changes, but it could be nicer...

./_source/plugins/stylescombo/plugins.js

onRender : function()
{
	function updateCombo( ev ) {
		var currentValue = this.getValue();
		var elementPath = ev.data.path,
		elements = elementPath.elements;

		// For each element into the elements path.
		for ( var i = 0, count = elements.length, element ; i < count ; i++ )
		{
			element = elements[i];

			// Check if the element is removable by any of
			// the styles.
			for ( var value in styles )
			{
				if ( styles[ value ].checkElementRemovable( element, true ) )
				{
					if ( value != currentValue )
						this.setValue( value );
					return;
				}
			}
		}

		// If no styles match, just empty it.
		this.setValue( '' );
	}

	editor.on( 'selectionChange', updateCombo, this);
	editor.on( 'removeFormat', updateCombo, this);
},

./_source/plugins/stylescombo/plugins.js (line 120)

	editor.getSelection().selectRanges( ranges );
	editor.fire( 'removeFormat', { path : new CKEDITOR.dom.elementPath( editor.getSelection().getStartElement() ) } );

Sample URLs:

• Bug behavior ​http://ckeditor.webunity.nl/index-3-4-2-nocontainer.php
  
• Correct behavior ​http://ckeditor.webunity.nl/index-3-4-2-container.php
  
• Patched behavior ​http://ckeditor.webunity.nl/index-3-4-2-patched.php
  

[Garry Yao]

Much simplified reproducing steps here:

1. Open "Styles" combo and apply the "Deleted Text" style to any text selection;
2. Click on "Remove Format" button, note that inline label is changed on "Styles" combo, then open the combo again;

• Actual: "Deleted Text" option keep marked on the list.

[Garry Yao]

#6500 and #7032 has been marked as DUPs.

[Garry Yao]

Fixed with [6453].

[Garry Yao]

Reopen because the fix is reverted by #7539.

[Garry Yao]

Fixed with [6946].

[Garry Yao]

The tc from first comment is still reproducible on trunk.

[Garry Yao]

Confirmed to be regressed by Firefox since version 7.

[Jakub Ś]

Still reproducible in Firefox 11

[Jakub Ś]

This issue is no longer reproducible in latest CKEditor 4.4.7 and FF 37.0.1