Ticket #3744 (closed defect: fixed)

Opened 2 years ago

Last modified 9 months ago

required attribute "alt" not specified.

Reported by: awallin Assigned to:
Priority: high Milestone: 2.5
Component: TinyMCE Version: 2.1
Severity: normal Keywords: dev-feedback
Cc: foolswisdom

Description

When I write posts in wp (using the visual editor) and insert images into my posts I usually don't bother inserting an ALT text. AFAIK an empty ALT attribute should be valid XHTML. However, I find that WP2.1 creates posts for me with _no_ ALT tag, and that results in a lot of errors when I do XHTML validation. My site is at www.anderswallin.net if you want to try this.

Change History

02/03/07 18:33:09 changed by foolswisdom

awallin, thank you for the detailed bug report!

Relates to #3100.

02/03/07 19:00:04 changed by awallin

#3100 mentios upload via the admin panel. The behaviour then seems to be that ALT is set to the filename.

I usually upload my images with a separate ftp client, and insert the pictures using the image icon of the visual editor. I type in the url of the image and click OK. This should set the ALT attribute to an empty string, and not leave out the ALT attribute altogether.

02/21/07 15:04:59 changed by Nazgul

  • keywords changed from images ALT XHTML to images ALT XHTML TinyMCE dev-fedback.

2.1.1 is out, so bumping the milestone to 2.1.2.

Should TinyMCE bugs be fixed here or reported upstream?

02/21/07 15:05:43 changed by Nazgul

  • milestone changed from 2.1.1 to 2.1.2.

02/22/07 07:01:29 changed by rob1n

  • component changed from General to TinyMCE.

03/28/07 00:10:56 changed by foolswisdom

  • milestone changed from 2.1.3 to 2.3.

06/09/07 21:27:59 changed by cpoteet

  • owner changed from anonymous to cpoteet.
  • status changed from new to assigned.

06/10/07 02:44:23 changed by cpoteet

  • cc set to foolswisdom.
  • keywords changed from images ALT XHTML TinyMCE dev-fedback to dev-fedback.
  • status changed from assigned to closed.
  • resolution set to invalid.

There is an image description box in the TinyMCE dialog box. It seems to be user error. You can't expect the machine to do everything for you :)

(follow-up: ↓ 10 ) 06/11/07 03:40:34 changed by rob1n

  • status changed from closed to reopened.
  • resolution deleted.

But what if the user doesn't want to add in a description? IMO, TinyMCE should correct for this and use at least a blank alt="" attribute.

(in reply to: ↑ 9 ) 06/11/07 11:56:19 changed by JeremyVisser

Replying to rob1n:

But what if the user doesn't want to add in a description? IMO, TinyMCE should correct for this and use at least a blank alt="" attribute.

+1 for that suggestion.

06/11/07 11:56:25 changed by JeremyVisser

  • keywords changed from dev-fedback to dev-feedback.

06/11/07 14:21:07 changed by markjaquith

Reluctant +1

The point of the blank alt="" is to designate images that don't need alternative representation. Usually that means background images or images used to break up content. alt="" shouldn't act as a crutch for people who are too lazy to provide an alternative representation for their images. By doing this for them, we're validating their laziness by hiding the symptom. That said, WordPress should publish valid code out of the box. Let the accessibility validators ding them for all the blank alt=""

(follow-up: ↓ 14 ) 06/11/07 14:37:58 changed by westi

I am also reluctant to support this from the accessibility point of view.

If it is to go in we need to push it upstream as it is a change to TinyMCE and if they will make the change then it will get pulled into the next version of WP as we update TinyMCE.

(in reply to: ↑ 13 ) 07/21/07 16:42:39 changed by Nazgul

Replying to westi:

I am also reluctant to support this from the accessibility point of view.

An empty alt tag is better than none, because if there is no alt tag present, most screenreaders will speak the filename. That doesn't sound very accessible to me.

09/12/07 17:59:46 changed by ryan

  • milestone changed from 2.3 to 2.4 (next).

10/05/07 19:21:06 changed by cpoteet

  • owner deleted.
  • status changed from reopened to new.

I think this is a TinyMCE issue and not a WP issue.

03/06/08 11:58:08 changed by azaozz

  • status changed from new to closed.
  • resolution set to fixed.

TinyMCE 3 inserts empty alt if nothing is entered in the field.