Opened 18 years ago

Closed 18 years ago

#4652 closed enhancement (invalid)

Added alt="xyz" text to various links

Reported by: Yeechang Lee <ylee@…> Owned by: xris
Priority: minor Milestone: 0.22
Component: mythweb Version: head
Severity: medium Keywords: mythweb
Cc: Ticket locked: no

Description

The attached patch adds alt="xyz" tags to various graphics in MythWeb, making pages more attractive-looking in text-based browsers like Elinks.

Attachments (3)

patch-mythwebalttext.diff​ (11.9 KB ) - added by Yeechang Lee <ylee@…> 18 years ago.
patch-mythwebalttext2.diff​ (11.9 KB ) - added by Yeechang Lee <ylee@…> 18 years ago.
Version with &gt; and &lt; instead of > and <.
patch-mythwebalttext3.diff​ (12.2 KB ) - added by Yeechang Lee <ylee@…> 18 years ago.
More alt text rewrites/additions.

Download all attachments as: .zip

Change History (10)

by Yeechang Lee <ylee@…>, 18 years ago

Attachment: patch-mythwebalttext.diff​ added

comment:1 by xris, 18 years ago

Status: new → infoneeded_new

Please resubmit with proper html entity escaping.

by Yeechang Lee <ylee@…>, 18 years ago

Attachment: patch-mythwebalttext2.diff​ added

Version with &gt; and &lt; instead of > and <.

by Yeechang Lee <ylee@…>, 18 years ago

Attachment: patch-mythwebalttext3.diff​ added

More alt text rewrites/additions.

comment:2 by xris, 18 years ago

Please make patches with svn diff against current svn trunk.

comment:3 by Rob Smith, 18 years ago

Milestone: unknown → 0.22
Status: infoneeded_new → new

comment:4 by xris, 18 years ago

Status: new → infoneeded_new

comment:5 by anonymous, 18 years ago

Can these tags be localized?

comment:6 by xris, 18 years ago

they should be, yes... and alt="" for the lite templates, and title="" for the default template (to create tooltips). to localize, use: alt="<?php echo t('text') ?>"

comment:7 by stuartm, 18 years ago

Resolution: → invalid
Status: infoneeded_new → closed

Requested updated patches not provided, closing.

Note: See TracTickets for help on using tickets.