413661 - Addons manager: "Show Information" and "Skip" buttons have the same accesskey

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, defect)

Description

[Vlado Valastiak [:wladow] @ Mozilla.sk]

Attachment: change Skip button accesskey

Show information and Skip buttons have the same accesskey, let's change Skip accesskey to "k"

Comment 1

[Dave Townsend [:mossop] (Out until 28th September)]

Comment on attachment 298713 [details] [diff] [review]
change Skip button accesskey

r=mossop, thanks. Check in a.s.a.p or mark as checkin-needed if you aren't able to.

Comment 2

[Dave Townsend [:mossop] (Out until 28th September)]

Comment on attachment 298713 [details] [diff] [review]
change Skip button accesskey

From Pike over IRC, no approval is needed currently

Comment 3

[Dave Townsend [:mossop] (Out until 28th September)]

Checking in extensions.dtd;
/cvsroot/mozilla/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.dtd,v  <--  extensions.dtd
new revision: 1.29; previous revision: 1.28
done

Comment 4

[Vlado Valastiak [:wladow] @ Mozilla.sk]

Dave, see this: http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/toolkit/locales/en-US/chrome/mozapps/extensions&command=DIFF_FRAMESET&file=extensions.dtd&rev1=1.28&rev2=1.29&root=/cvsroot 

Comment 5

[Axel Hecht [:Pike]]

Dave, I don't think you had a clean tree when landing this, back the accidental changes out?

Comment 6

[Reed Loden [:reed] (use needinfo?)]

(In reply to comment #5)
> Dave, I don't think you had a clean tree when landing this, back the accidental
> changes out?

Checking in toolkit/locales/en-US/chrome/mozapps/extensions/extensions.dtd;
/cvsroot/mozilla/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.dtd,v  <--  extensions.dtd
new revision: 1.30; previous revision: 1.29
done