[pl] make default width of pageInfo dialog a tad wider

RESOLVED FIXED

Status

Mozilla Localizations
pl / Polish
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Pike, Assigned: stef)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Looking at http://pike.github.io/firefox-l10n-tests/?file=security%2Fdv-details.png&locale=pl, the pageinfo dialog should get a tad wider default width.

The width of that dialog is persisted, so you need to test on a new window. My suspicion is that 620 is OK.

Way to test: Open firefox with a new profile, enable remote debugging, use window.resizeBy(20,0) to verify, switch through all tabs to confirm.

Yeah, that's pretty naughty, and it took me a while to find this for another locale. Which looked similarily cut off and 620 was OK there.
(Assignee)

Comment 1

2 years ago
I don't see it on OS X 10.11 and 42 beta but I do on 44 nightly.
Deleting xulstore.json from profile seems to remove remembered window size.

Could anyone test this under Windows 10 (and optionally 7) for 42 and 44 and tell me how much resizing is needed there?
Assignee: nobody → splewako
(Assignee)

Comment 4

2 years ago
Created attachment 8680644 [details]
1215951.png

Firefox 42, https://www.google.pl, TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256, pageInfoWindow.width set to 600 px
(Assignee)

Comment 5

2 years ago
On Windows 7 even 640 px is not enough to have shortened pageInfo_Privacy_Encrypted2 fit into two lines.

https://hg.mozilla.org/l10n-central/pl/rev/6410f42a6d8b sets window size to 650x500 making pageInfo_Privacy_Encrypted2 two lines, making all fit (tested on Windows 7, OS X 10.11) and giving us few pixels in reserve (both dimensions).
(Assignee)

Comment 6

2 years ago
Minimal window width seems to be related to cipher used and https://www.google.pl for example has one with longer name (TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256) than tested on http://pike.github.io/firefox-l10n-tests/?file=security%2Fdv-details.png&locale=de pushing things right and making situation even worse.

Would it be possible to use site with longer/longest cipher name in firefox-l10n-tests?
Flags: needinfo?(l10n)
(Assignee)

Comment 8

2 years ago
Looks good to me on Windows 7 and OS X El Capitan.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
(Reporter)

Comment 9

2 years ago
(In reply to Stefan Plewako [:stef] from comment #6)
> Minimal window width seems to be related to cipher used and
> https://www.google.pl for example has one with longer name
> (TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256) than tested on
> http://pike.github.io/firefox-l10n-tests/?file=security%2Fdv-details.
> png&locale=de pushing things right and making situation even worse.
> 
> Would it be possible to use site with longer/longest cipher name in
> firefox-l10n-tests?

I'm building on top of Henrik's suite, forwarding the question to him. Do you know?
Flags: needinfo?(l10n)
If you need another page for l10n testing you can make the modifications in your own test. We cannot change it for the base firefox-ui-tests given that we do not want to use remote websites. mozqa.com is located in scl3 and we can access internally without using a proxy. I don't know about the security stuff but maybe we could setup another subdomain with a longer cipher.
(Assignee)

Comment 11

2 years ago
Just had an opportunity to test this on Windows 10 and it is even worse… :(
At the size that was good for Windows 7, in 10 the window is not high enough and help button was cut.

https://hg.mozilla.org/l10n-central/pl/rev/8a5c37228e6b
You need to log in before you can comment on or make changes to this bug.