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.
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..
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 hurry. 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 320541 [details] [diff] [review] v1 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.
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] v1 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] v1 a+ schrep per dev.planning thread to land on CVS trunk.
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 <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 done
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.