Closed Bug 1107759 Opened 11 years ago Closed 11 years ago

Support resizing b2g desktop window

Categories

(Firefox OS Graveyard :: Runtime, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file, 3 obsolete files)

Today, when we resize b2g desktop, the system app keeps the same size. Ideally gaia would stretch to the top level window size.
Assignee: nobody → poirot.alex
Attached patch patch (obsolete) — Splinter Review
Fix window resizing and replace dpi support with pixel ratio. It looks like existing DPI support doesn't do anything... So instead of keeping a broken feature, let's just make something useful.
Attachment #8532298 - Flags: review?(fabrice)
The ratio features allows opening a scaled down TV build with a big resolution: b2g-bin --screen 1920x1080@0.25 This features can later be used by simulator profiles to control screen size and pixel ratio from WebIDE.
Alex, I cannot use this in my mac box. But it works well in my ubuntu box.
Comment on attachment 8532298 [details] [diff] [review] patch Review of attachment 8532298 [details] [diff] [review]: ----------------------------------------------------------------- Works fine for me too on linux. Can you file a follow-up to get the MacOS issue fixed?
Attachment #8532298 - Flags: review?(fabrice) → review+
Actually, while testing on Mac it appeared to me that it is also clunky on Linux, when the orientation is locked things go wrong. It get reseted to the default screen size with touch even position issues.
Attached patch patch v2 (obsolete) — Splinter Review
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=02b2a65f4f72 New patch that watch manual window resizes in order to update width/height attributes in screen.js. I also added a line: GlobalSimulatorScreen.mozOrientation = GlobalSimulatorScreen.screenOrientation = defaultOrientation; That should help with default orientation being different than portrait. John, Could you confirm it works better on Mac and also that the default orientation on TV is better handled?
Attachment #8532298 - Attachment is obsolete: true
Attachment #8537224 - Flags: review?(fabrice)
Attachment #8537224 - Flags: feedback?(im)
Attachment #8537224 - Flags: review?(fabrice) → review+
Comment on attachment 8537224 [details] [diff] [review] patch v2 Review of attachment 8537224 [details] [diff] [review]: ----------------------------------------------------------------- The orientation and resize are working in Mac. Looks pretty good. I found that I was wrong at comment 3. The resize worked at that time. But I use TV build to test it. The mulet's orientation is wrong and I change the default orientation. While I resized the screen, it only shows partial screen. So, I feel it doesn't work. Sorry about this false alarm.
Attachment #8537224 - Flags: feedback?(im) → feedback+
Another try, rebased, to see if the failure are really related to this patch: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=3004e88484f6
Looks like it wasn't...
Keywords: checkin-needed
I have another patch on top of this one to fix mulet rotation.
Keywords: leave-open
Attached patch Fix mulet (obsolete) — Splinter Review
I had to add these two tweaks to get it working nicely also on Mulet...
Attachment #8540684 - Flags: review?(fabrice)
Attachment #8540684 - Flags: review?(fabrice) → review+
Needs rebasing.
Keywords: checkin-needed
Attached patch patch v3Splinter Review
Rebased and merged the two patches.
Attachment #8537224 - Attachment is obsolete: true
Attachment #8540684 - Attachment is obsolete: true
Attachment #8540957 - Flags: review+
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: