Bug 1535253 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Since bug 1493844, many dialog windows/prompts are missing padding which makes them look really poor and unpolished. I'm not sure of the common theme between them but maybe Jared knows (I think he reviewed the one of two patches [part 2/3] that causes this) or understand what the general fix would be.

* passwordManager.xul (when opened in its own window from the context menu or autocomplete footer)
* changemp.xul - Master password: "Password change succeeded" - Uses Services.prompt.alert
* ReportFalseDeceptiveSite() - Uses Services.prompt.alert
* Set as homepage confirmation - Upon URL drag and drop on the home button - Services.prompt.confirmEx
* etc.

[Tracking Requested - why for this release]: Secondary UI regression affecting the password manager dialog.
Since bug 1493844, many dialog windows/prompts are missing padding which makes them look really poor and unpolished. I'm not sure of the common theme between them but maybe Jared knows (I think he reviewed the one of two patches [part 2/3] that causes this) or understand what the general fix would be.

* passwordManager.xul (when opened in its own window from the context menu or autocomplete footer)
* changemp.xul - Master password: "Password change succeeded" - Uses Services.prompt.alert => commonDialog.xul
* ReportFalseDeceptiveSite() - Uses Services.prompt.alert
* Set as homepage confirmation - Upon URL drag and drop on the home button - Services.prompt.confirmEx
* etc.

[Tracking Requested - why for this release]: Secondary UI regression affecting the password manager dialog.
Since bug 1493844, many dialog windows/prompts are missing padding which makes them look really poor and unpolished. I'm not sure of the common theme between them but maybe Jared knows (I think he reviewed the one of two patches [part 2/3] that causes this) or understand what the general fix would be.

* passwordManager.xul (when opened in its own window from the context menu or autocomplete footer)
* ~~changemp.xul - Master password: "Password change succeeded" - Uses Services.prompt.alert => commonDialog.xul~~
* * Existing problem before bug 1493844
* ReportFalseDeceptiveSite() - Uses Services.prompt.alert
* Set as homepage confirmation - Upon URL drag and drop on the home button - Services.prompt.confirmEx
* etc.

[Tracking Requested - why for this release]: Secondary UI regression affecting the password manager dialog.

Back to Bug 1535253 Comment 0