358120 - add id attributes to passwordManager.xul

Status: RESOLVED FIXED

(Toolkit :: Password Manager, defect)

Description

[Michael Newton]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0

Adding a new button to the password manager dialog is impossible; the hbox containing the buttons has no id

Reproducible: Always

Comment 1

[Martijn Wargers (dead)]

Attachment: patch

So you want something like this, I guess?

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 243634 [details] [diff] [review]
patch

I'd use "SignonViewerButtons" to be consistent with the other IDs in this file. r=me with that.

Comment 3

[Martijn Wargers (dead)]

Attachment: patch for check-in

Ok, used SignonViewerButtons now.

Comment 4

[Martijn Wargers (dead)]

Checking in toolkit/components/passwordmgr/resources/content/passwordManager.xul
;
/cvsroot/mozilla/toolkit/components/passwordmgr/resources/content/passwordManage
r.xul,v  <--  passwordManager.xul
new revision: 1.12; previous revision: 1.11
done