Insert Smiley does not include required alt attribute

[Mike Koepke]

When inserting a smiley, the editor does not include the required alt attribute. This does not impact functionality but causes html validation errors. Ideally it just needs to add alt="" when inserting.

[Frederico Caldeira Knabben]

The editor automatically adds a alt="" to images with no alt.

I've just tested the smileys with both IE6 and FF2, with these steps:

1. Insert and smiley;
2. Switch to source.

The alt="" has been correctly included in the <img /> tag.

Do you have more info regarding the bug?

[Mike Koepke]

I started with a blank textarea. If I click the Smiley button and added the smile smiley, here is what gets generated.

<p><img src="./fckeditor/editor/images/smiley/msn/regular_smile.gif" /></p>

This is using 2.4. Note that there is no alt=""

Mike

[Frederico Caldeira Knabben]

Hi Mike... just to be 100% sure we are doing the same thing... Did you do the exactly same steps I've mentioned in my last comment? Do you have the same wrong results with the ​nightly build? What is your browser version?

This is what I'm having instead, with both IE6 and FF2:

<p><img src="/fckeditor/editor/images/smiley/msn/regular_smile.gif" alt="" /></p>

[Mike Koepke]

I haven't tried the nightly build yet. I'm using FF 2.0.0.2. Using IETAB, I switched to IE7 engine and had the same issue.

I will say I have FCK embedded into a Wordpress plugin with a few others plugins added. Let me try and troubleshoot this some more to see if something else is causing a conflict.

Mike

[Frederico Caldeira Knabben]

I meant, can you reproduce it in the nightly build test page at: ​http://www.fckeditor.net/nightly/fckeditor/_samples/default.html

You don't need to download and install it.

[Mike Koepke]

This works correctly with the nightly build. Close this ticket as unreproduceable.

[Frederico Caldeira Knabben]

Great! Thanks for the update.