bsharma Reporter
	Description • 24 years ago

The accesskey attribute does not work for the area element although the area is 
accessible through mouse and it works fine.

	Christine Hoffman
	Comment 1 • 23 years ago

*** Bug 84357 has been marked as a duplicate of this bug. ***

	Heikki Toivonen (remove -bugzilla when emailing directly)
	Comment 2 • 23 years ago

HTML Element component is deprecated, and should be removed from Bugzilla.
Clayton is not the correct owner for these. Reassigning to layout.

	bsharma Reporter
	Comment 3 • 23 years ago

Build 2001-07-27-branch on WinNT and Linux.

Same is the problem with the test case of no DOCTYPE.

	Moied
	Comment 4 • 23 years ago

Tested with build ID 2001-09-21-05-0.9.4 on windows 2000 SP1, same problem, 
Changing OS to ALL

	anthonyd
	Comment 5 • 23 years ago

confirmed in today's build, but also not a table specific problem, reassigning 
to core owner.

	rubydoo123
	Comment 6 • 23 years ago

Bumping this one up since it affects accessibility. Confirmed using the win98 
build from 20011127.

	Moied
	Comment 7 • 23 years ago

*** Bug 49968 has been marked as a duplicate of this bug. ***

	Kevin McCluskey (gone)
	Comment 8 • 23 years ago

Marking nsbeta1-. Engineers are overloaded with higher priority bugs, so there
is not much chance we will be able to get to this.

	Boris Zbarsky [:bzbarsky]
	Comment 9 • 22 years ago

*** Bug 145394 has been marked as a duplicate of this bug. ***

	Bolian Yin Assignee
	Comment 10 • 22 years ago

=> bolian.
I am trying to fix it.

	Bolian Yin Assignee
	Comment 11 • 22 years ago

seek r=?

	David Baron :dbaron:
	Comment 12 • 22 years ago

Comment on attachment 90458 [details] [diff] [review]
set accesskey attribute of area

I'd think you'd need to change SetDocument as well, don't you?	I'm surprised
this works, except for the case of dynamic attribute changes.  See the code in
nsHTMLAnchorElement.cpp.

	David Baron :dbaron:
	Comment 14 • 22 years ago

Comment on attachment 90597 [details] [diff] [review]
patch_v2 (add SetDocument method)

r=dbaron.  You should probably get jst to sr, though.

	Johnny Stenback (:jst)
	Comment 15 • 22 years ago

Comment on attachment 90597 [details] [diff] [review]
patch_v2 (add SetDocument method)

+  if (aName == nsHTMLAtoms::accesskey && kNameSpaceID_None == aNameSpaceID) {

Please change the latter equality check to do variable == constant in stead of
the other way around (i.e. aNameSpaceID == kNameSpaceID_None in stead of
kNameSpaceID_None == aNameSpaceID).

sr=jst

	Asa Dotzler [:asa]
	Comment 17 • 22 years ago

Comment on attachment 90885 [details] [diff] [review]
patch (for checkin, add jst's comment)

a=asa (on behalf of drivers) for checkin to 1.1

	Kyle Yuan
	Comment 19 • 22 years ago

checked into trunk!

	Moied
	Comment 20 • 22 years ago

we would like to see this fix checked in branch too, removing nsbeta1- and
adding nsbeta1

	dsirnapalli
	Comment 21 • 22 years ago

We moved the test cases to different location. I updated the above url to
reflect the current test case.
I was verifying this bug in branch build 20030212 and found its not checked into
the branch yet. Can anybody verify if its checked into branch.