When DPI is 144 or more, CSS px units are used to report: screen.width screen.height screen.availWidth screen.availHeight while screen px units are used to report: window.outerWidth window.outerHeight window.innerWidth window.innerHeight The same units should be used for all. I recommend screen px units for the foreseeable future at least. cf. bug 375519
Created attachment 310996 [details] screenshot of subject URL demonstrating inconsistency FF 22.214.171.124 vs. FF 3.0b4 on OpenSUSE 10.2 in KDE @ 1920x1440
Pretty much anything exposed to content should probably be reporting CSS px, since any sizing content does will be in CSS px...
Yeah, I think we should fix outer* and inner*
Hmm, the code seems to indicate innerWidth and innerHeight are returned in CSS pixels. Not sure what's up with that. outerWidth and outerHeight should be easy to fix.
Created attachment 312006 [details] testcase Roc, you're right. Again I was derelict in a comment 0. I'm guessing that I was going on ancient memory of 1.5a, maybe before the bug 375519 checkin, when both inners & outers matched in device px. Currently only the outers are in device px. On Linux at least, desktop taskbar size is not being subtracted from screen.avail*. e.g. screen.height = screen.availHeight.
Blocking as not fixing this now would make it significantly harder to fix later if people start depending on this broken behavior.
In up to date Kubuntu Hardy's FF 3.0b4 2008031317, all are displaying only device px. It looks and works just like branch. It too is not subtracting taskbar size from avail.
Created attachment 312722 [details] [diff] [review] fix Fixes getting and setting of outerWidth/outerHeight.
Created attachment 312723 [details] testcase I'm not sure if this can be automated. We'd have to change layout.css.dpi on the fly which sounds scary.
9 years ago
9 years ago
Perhaps worth adding some sort of single-test harness or something, then, but we should still get this automated. Somehow. Eventually.