93083 - Warning dialog "Certificate from unrecognized CA" is too small

Status: VERIFIED FIXED

(Core Graveyard :: Security: UI, defect, P2)

Description

[doctor__j]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.2+) Gecko/20010730
BuildID:    2001073006

Reproducible: Always
Steps to Reproduce:
1. Load the above URL.
2. See the three buttons of the warning dialog clipped at bottom.

Comment 1

[doctor__j]

Attachment: Screenshot

Comment 2

[John Unruh]

Confirming on 8/1 WinNT trunk build. "www.hongkongpost.gov.hk" is long enough to 
make the sentence wrap to another line, thus putting the buttons off screen.

Comment 3

[Stephane Saux]

*** This bug has been marked as a duplicate of 81958 ***

Comment 4

[Stephane Saux]

Reopening
->Rangan
The new dialog from bug 91466 needs the proper fix to make it work regardless of
screen resolution and font choices.

cc [e-mail] for comment.

Comment 5

[Stephane Saux]

->rangansen

Comment 6

[John Unruh]

Mass assigning QA to ckritzer.

Comment 7

[John Unruh]

*** Bug 93134 has been marked as a duplicate of this bug. ***

Comment 8

[Rangan Sen]

Here is a suggested patch - This is also caused by bug# 91137, this patch uses
the temp workaround fix described in bud# 94384.
This fix, in some cases, might make the window a bit too long, leaving some
amount of space below the ok/cancel buttons.

Comment 9

[Rangan Sen]

Attachment: patch

Comment 10

[Javier Delgadillo]

r=javi (as workaround)

Comment 11

[Erb]

Similar code for domainMismatch.js in onLoad:


  var windowReference = document.getElementById('domainMismatch');
  var wdth = window.innerWidth; // THIS IS NEEDED,
  window.sizeToContent();
  windowReference.setAttribute("width",window.innerWidth + 30);
  var hght = window.innerHeight; // THIS IS NEEDED,
  window.sizeToContent();
  windowReference.setAttribute("height",window.innerHeight + 40);

Comment 12

[Stephen Walker]

*** Bug 94516 has been marked as a duplicate of this bug. ***

Comment 13

[Joe Hewitt (gone)]

sr=hewitt

Comment 14

[Rangan Sen]

patch checked in

Comment 15

[Håkan Waara]

Sorry for not being responsive, I was on vacation.

Thanks for fixing this rangan.

Comment 16

[John Unruh]

Verified.