Closed
Bug 138028
Opened 23 years ago
Closed 23 years ago
accesskey="&more.label;" should be accesskey="&more.accesskey;"
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.1beta
People
(Reporter: ruixu, Assigned: nhottanscp)
References
Details
(Keywords: intl, Whiteboard: [adt3 rtm])
Attachments
(1 file)
799 bytes,
patch
|
naving
:
review+
Bienvenu
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
This is separated from bug http://bugscape.netscape.com/show_bug.cgi?id=13834, and need more fix in general.
As per rchen's comments in bugscape bug 13834:
http://lxr.mozilla.org/seamonkey/source/mailnews/base/search/resources/content/s
earchTermOverlay.xul#79
accesskey="&more.label;" should be accesskey="&more.accesskey;"
It caused shortcut issue on localized build. -> nsbeta1
Keywords: nsbeta1
Comment 2•23 years ago
|
||
Michele, we need to plus this low risk bug fix.
Assignee | ||
Comment 5•23 years ago
|
||
Assignee | ||
Comment 6•23 years ago
|
||
cc to naving, could you review the patch?
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.1beta
Comment 7•23 years ago
|
||
Comment on attachment 83388 [details] [diff] [review]
Set a correct accesskey value for "more".
r=naving
Attachment #83388 -
Flags: review+
Comment 8•23 years ago
|
||
Comment on attachment 83388 [details] [diff] [review]
Set a correct accesskey value for "more".
sr=bienvenu
Attachment #83388 -
Flags: superreview+
Assignee | ||
Comment 10•23 years ago
|
||
checked in to the trunk
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 11•23 years ago
|
||
rui- can you verified this on trunk?
Comment 12•23 years ago
|
||
adding adt1.0.0+. Please get drivers approval and then checkin to the 1.0 branch.
Comment 13•23 years ago
|
||
Comment on attachment 83388 [details] [diff] [review]
Set a correct accesskey value for "more".
a=rjesup@wgate.com for drivers; please check into branch asap
Attachment #83388 -
Flags: approval+
Reporter | ||
Comment 15•23 years ago
|
||
Verified on trunk.
Comment 16•23 years ago
|
||
I don't have a localized build. Who can verify it on Branch?
Comment 17•23 years ago
|
||
You don't need a localized build to verify it on Branch. Just go to the dtd
file and modify the entity "more.accesskey" to some other character (other than
the first letter of more.label) and see if it reflects the change.
Reporter | ||
Comment 18•23 years ago
|
||
Verified on branch 20020521.
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.0 → verified1.0.0
Updated•23 years ago
|
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•