Closed Bug 479307 Opened 14 years ago Closed 14 years ago

sizemode="maximized" does not work well on desktop

Categories

(Firefox for Android Graveyard :: General, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file, 1 obsolete file)

We changed to use sizemode="maximized" so the window filled the device screen. This doesn't work so well on desktop.

Patch uses sizemode for mobile platforms and a fixed width/height for desktops
Attached patch patch (obsolete) — Splinter Review
hmm, the patch didn't make it the first time
Assignee: nobody → mark.finkle
Attachment #363248 - Flags: review?(gavin.sharp)
Attachment #363248 - Attachment is patch: true
Attachment #363248 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 363248 [details] [diff] [review]
patch

>diff --git a/chrome/content/browser.xul b/chrome/content/browser.xul

>+#ifdef WINCE || MOZ_PLATFORM_HILDON

Does this "||" actually work? I haven't seen this syntax used for the preprocessor anywhere else, and see no mention of it in the current python script/test or the old perl preprocessor docs.
Uses the correct conditionals. Gavin is correct, no complex conditional tests are supported. Dumb luck made my build work as expected.
Attachment #363248 - Attachment is obsolete: true
Attachment #363322 - Flags: review?(gavin.sharp)
Attachment #363248 - Flags: review?(gavin.sharp)
Attachment #363322 - Flags: review?(gavin.sharp) → review+
http://hg.mozilla.org/mobile-browser/rev/d7c72600678e
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
verified fixed in 20091001 1.9.2 b4 on my mac 10.5
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.