Closed Bug 483960 Opened 12 years ago Closed 12 years ago
Password Manager opens a window, not a dialog
When toPasswordManager was originally written, toOpenWindowByType only opened windows, since it hardcoded the window features, although the password manager is really a dialog. Fortunately (or unfortunately, as toOpenWindowByType was enhanced before toPasswordManager was checked in) toOpenWindowByType now supports an optional features parameter, which we can use to make the password manager open with the correct chrome and resizable features.
Also fix the type so we can find it if it's already open.
Comment on attachment 368022 [details] [diff] [review] Make this be a resizable dialog instead of an app window. Ah, good catch. Are you going to remove the chrome feature from our other openDialog calls? ;-)
It'd make a nice "getting started" bug for someone. There's so much crap out there. "dialog=yes", "modal=yes" (the "=yes" part is redundant), "all,x,y,z" where x, y, and z are covered by all.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.