433298 - EULA cut off checkbox/buttons on bottom - unusable on small (EeePC) screens

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

[Ed Lee :Mardak]

Bug 431713 revved the EULA number, so it popped up when I tried upgrading beta 5 release to rc1.

The EeePC screen doesn't fit the whole EULA window, and even if you resize it, the checkbox and buttons aren't visible.

I had to ctrl-shift-tab from the beginning and hit space and enter until it was happy.

Comment 1

[Ed Lee :Mardak]

chrome://browser/content/EULA.xul

http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/browser/base/content/EULA.xul&rev=1.1&mark=69#69

<iframe style="min-height: 30em; min-width: 105ex" id="EULATextFrame" type="content" flex="1" src=""/>

Either reduce the min-height or get rid of it? But without it, it's possible that nothing shows up if the window is really really short..

Comment 2

[Mike Beltzner [:beltzner, not reading bugmail]]

This isn't a regression, though the EeePC wasn't around when Firefox 2 was released. A good ride-along candidate, but I'm not sure it's an outright blocker on the release. If we do not block on this, we definitely need to fix it for the first branch release before we do major update.

Axel: is there an l10n impact to removing min-height?

Comment 3

[Axel Hecht [:Pike]]

I don't think there's an l10n impact here, just a lot of "did you test on this screen and that platform".

Comment 4

[aja+bugzilla]

Thanks, Mardak. Noticed this the other nite, but hadn't got around to filing
yet. Figured it was probably too late to block anyway, so there was no real
hurry.

If/when this is accepted, there's some minor probs with preferences menus that
probably should be filed as well.

Comment 5

[Ed Lee :Mardak]

Well, as it stands right now, unless the EeePC user is clever enough to figure out how to get past the EULA without seeing the checkbox and button, s/he won't be able to use Firefox 3 (RC1).

On the EeePC I'm using.. changing the original "min-height: 30em; min-width: 105ex" to 18em and 95ex fits the screen.

Comment 6

[Ed Lee :Mardak]

Attachment: screenshot of 18em 95ex

Comment 7

[Ed Lee :Mardak]

Attachment: sceenshot of trunk (30em 105ex)

Comment 8

[Ed Lee :Mardak]

Attachment: v1

Quick fix for 30em -> 18em and 105ex -> 95ex.

Comment 9

[Ed Lee :Mardak]

For RC1, people can do alt-c then press enter.. at least for the english locale.

I believe it's ctrl-shift-tab, ctrl-shift-tab, up, enter to get things working in general. (focus to cancel, focus to radios, change selection to accept, ok for dialog)

Comment 10

[Mike Schroepfer]

eepc has a uncommonly tiny screen and there are ways for users to read the whole thing.  So we can't block RC1 for this.  If we have an RC2 we can take it there.

Comment 11

[Ed Lee :Mardak]

(In reply to comment #10)
> there are ways for users to read the whole thing
The problem isn't reading the EULA. It's interacting with the window because the minimum height of the text is taller than the window, so the accept/not accept radios are off the screen in addition to the ok/cancel button.

Even if the user hits the maximize window button at the top, radios and buttons aren't shown.

The only other visible things to click are close and minimize.

Comment 12

[Ed Lee :Mardak]

Like in attachment 320537 [details], if you click the maximize button at the top-right, it won't show anything different. Things aren't better even if you retract the taskbar at the bottom (black arrow at bottom-right).

You can drag the window around left/right/down, but it can't go any higher than the top of the screen.

Comment 14

[Tony Chung [:tchung]]

I am renom'ing this bug for blocking.   See comments in https://bugzilla.mozilla.org/show_bug.cgi?id=433627#c2 and https://bugzilla.mozilla.org/show_bug.cgi?id=433627#c3 for my arguments.  I think simply adding toolbars would fix things.

Comment 15

[Mike Beltzner [:beltzner, not reading bugmail]]

Let's get the patch reviewed and ready to go along with an RC2 if we do one, but I don't think this bug on its own requires that we spin another candidate. It'll be an easy take for 3.0.1, though, and we can try to recommend it for the patchsets that various Linux vendors include in their pre-installed versions.

Comment 16

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 320541 [details] [diff] [review]
v1

This makes it a bit harder to actually read the EULA on large screens, but who does that anyways? :)

Comment 17

[Mike Schroepfer]

Comment on attachment 320541 [details] [diff] [review]
v1

a+ schrep per dev.planning thread to land on CVS trunk.

Comment 18

[Ed Lee :Mardak]

Committed to both hg (mozilla-central) and cvs.

http://hg.mozilla.org/mozilla-central/index.cgi/rev/79924d3b5bba
remote: added 1 changesets with 1 changes to 1 files

changeset:   15143:79924d3b5bba
tag:         tip
user:        Edward Lee <[e-mail]>
date:        Fri May 23 02:17:25 2008 -0500
summary:     Bug 433298 - EULA cut off checkbox/buttons on bottom - unusable on small (EeePC) screens. r=gavin, a1.9=schrep

Checking in browser/base/content/EULA.xul;
/cvsroot/mozilla/browser/base/content/EULA.xul,v  <--  EULA.xul
new revision: 1.2; previous revision: 1.1
done

Comment 19

[Bartłomiej Brzozowiec (BartZilla)]

(In reply to comment #16)
> (From update of attachment 320541 [details] [diff] [review])
> This makes it a bit harder to actually read the EULA on large screens, but who
> does that anyways? :)
> 

I do. I think it sucks, especially item 4. (http://mxr.mozilla.org/firefox/source/browser/EULA#16):

"4.  PRIVACY POLICY.  You agree to the Mozilla Firefox Privacy Policy, made available online at http://www.mozilla.com/legal/privacy/, as that policy may be changed from time to time.  When Mozilla changes the policy in a material way a notice will be posted on the website at www.mozilla.com and when any change is made in the privacy policy, the updated policy will be posted at the above link.  It is your responsibility to ensure that you understand the terms of the privacy policy, so you should periodically check the current version of the policy for changes."

So, I guess it means that:
1. Privacy policy is integral part of the EULA (so Privacy policy should be available to the user during installation -- it isn't now if user is off-line)
and
2. user should read Privacy policy form time to time, so, according to EULA, user should do the following:
  - turn on computer
  - run another browser, without so ridiculous EULA
  - using another browser go to http://www.mozilla.com/legal/privacy/
  - read it carefully and compare with previous version
  - if it is acceptable, THEN finally run Firefox.

So, in summary, it sucks. Solution is simple: disable by default so called safebrowsing by Google, then you wouldn't need so ridiculous EULA. (Of course it is not going to happen, since Google really wants to "protect" users of FF by default, but hey, at least I can tell that I tried to do something with this...)

Comment 20

[:Gavin Sharp [email: [e-mail]]]

BartZilla, if you have problems with the EULA, please raise them in an appropriate forum (like the mozilla.dev.apps.firefox newsgroup). They're not related to this bug.

Comment 21

[Tony Chung [:tchung]]

Verified fix on Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9) Gecko/2008052912 Firefox/3.0.   EULA displays correctly on a 800x600 mode.