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.
<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..
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?
I don't think there's an l10n impact here, just a lot of "did you test on this screen and that platform".
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
If/when this is accepted, there's some minor probs with preferences menus that
probably should be filed as well.
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.
Created attachment 320535 [details]
screenshot of 18em 95ex
Created attachment 320537 [details]
sceenshot of trunk (30em 105ex)
Created attachment 320541 [details] [diff] [review]
Quick fix for 30em -> 18em and 105ex -> 95ex.
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)
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.
(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.
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.
*** Bug 433627 has been marked as a duplicate of this bug. ***
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.
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 on attachment 320541 [details] [diff] [review]
This makes it a bit harder to actually read the EULA on large screens, but who does that anyways? :)
Comment on attachment 320541 [details] [diff] [review]
a+ schrep per dev.planning thread to land on CVS trunk.
Committed to both hg (mozilla-central) and cvs.
remote: added 1 changesets with 1 changes to 1 files
user: Edward Lee <firstname.lastname@example.org>
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
(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):
So, I guess it means that:
- 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...)
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.
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.