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.
Created attachment 368022 [details] [diff] [review] Make this be a resizable dialog instead of an app window. 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
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.