Opened 12 years ago

Closed 12 years ago

#215 closed Bug (worksforme)

Insert Smiley does not include required alt attribute

Reported by: Mike Koepke Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.4
Keywords: WorksForMe Cc:

Description

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.

Change History (7)

comment:1 Changed 12 years ago by Frederico Caldeira Knabben

Keywords: WorksForMe Pending added
Priority: LowNormal

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?

comment:2 Changed 12 years ago by 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

comment:3 Changed 12 years ago by 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>

comment:4 Changed 12 years ago by 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

comment:5 Changed 12 years ago by 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.

comment:6 Changed 12 years ago by Mike Koepke

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

comment:7 Changed 12 years ago by Frederico Caldeira Knabben

Keywords: Pending removed
Resolution: worksforme
Status: newclosed

Great! Thanks for the update.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy