Add the 'alt' attribute to checkboxes in the 'Saved Searches' panel

RESOLVED FIXED in Bugzilla 3.0

Status

()

Bugzilla
Bugzilla-General
--
enhancement
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Frédéric Buclin, Assigned: Frédéric Buclin)

Tracking

2.23
Bugzilla 3.0
Bug Flags:
approval +

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

12 years ago
This attribute should contain the name of the saved search so that Selenium can easily track it. I don't care if shared queries have the same attribute as it doesn't need to be unique. And this is not a problem for Selenium as we have very specific query names.
(Assignee)

Comment 1

12 years ago
Created attachment 241583 [details] [diff] [review]
patch, v1

The W3C specs say that all <input> field may have an alt="..." attribute.
Attachment #241583 - Flags: review?(mkanat)

Comment 2

12 years ago
Comment on attachment 241583 [details] [diff] [review]
patch, v1

r=me by inspection (at request of lpsolit on IRC)
Attachment #241583 - Flags: review+
(Assignee)

Updated

12 years ago
Attachment #241583 - Flags: review?(mkanat)
(Assignee)

Comment 3

12 years ago
This patch is required by Selenium to run.
Status: NEW → ASSIGNED
Flags: approval?
what does this look in Lynx and eLinks, which do things with the alt tag?
Flags: approval? → approval+
still waiting for an answer to the question in comment 4
Flags: approval+

Comment 6

12 years ago
they don't seem to do anything at least given http://www.rsz.jp/userprefs.html

so this is fine.
(Assignee)

Comment 7

12 years ago
I second what timeless said. The alt attribute is not displayed in Lynx.
Flags: approval?
Flags: approval? → approval+
(Assignee)

Comment 8

12 years ago
Checking in template/en/default/account/prefs/saved-searches.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/account/prefs/saved-searches.html.tmpl,v  <--  saved-searches.html.tmpl
new revision: 1.10; previous revision: 1.9
done
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.