Closed
Bug 1107759
Opened 11 years ago
Closed 11 years ago
Support resizing b2g desktop window
Categories
(Firefox OS Graveyard :: Runtime, defect)
Firefox OS Graveyard
Runtime
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ochameau, Assigned: ochameau)
References
Details
Attachments
(1 file, 3 obsolete files)
|
10.22 KB,
patch
|
ochameau
:
review+
|
Details | Diff | Splinter Review |
Today, when we resize b2g desktop, the system app keeps the same size.
Ideally gaia would stretch to the top level window size.
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → poirot.alex
| Assignee | ||
Comment 1•11 years ago
|
||
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.
| Assignee | ||
Updated•11 years ago
|
Attachment #8532298 -
Flags: review?(fabrice)
| Assignee | ||
Comment 2•11 years ago
|
||
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.
Comment 3•11 years ago
|
||
Alex,
I cannot use this in my mac box. But it works well in my ubuntu box.
Comment 4•11 years ago
|
||
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+
| Assignee | ||
Comment 5•11 years ago
|
||
| Assignee | ||
Comment 6•11 years ago
|
||
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.
| Assignee | ||
Comment 7•11 years ago
|
||
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)
Updated•11 years ago
|
Attachment #8537224 -
Flags: review?(fabrice) → review+
Comment 8•11 years ago
|
||
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+
| Assignee | ||
Comment 9•11 years ago
|
||
Another try, rebased, to see if the failure are really related to this patch:
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=3004e88484f6
| Assignee | ||
Comment 11•11 years ago
|
||
I have another patch on top of this one to fix mulet rotation.
Keywords: leave-open
| Assignee | ||
Comment 12•11 years ago
|
||
I had to add these two tweaks to get it working nicely also on Mulet...
Attachment #8540684 -
Flags: review?(fabrice)
| Assignee | ||
Comment 13•11 years ago
|
||
Try for the second patch: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=2d7b915bff17
Updated•11 years ago
|
Attachment #8540684 -
Flags: review?(fabrice) → review+
Updated•11 years ago
|
Keywords: leave-open
| Assignee | ||
Comment 15•11 years ago
|
||
Rebased and merged the two patches.
Attachment #8537224 -
Attachment is obsolete: true
Attachment #8540684 -
Attachment is obsolete: true
Attachment #8540957 -
Flags: review+
| Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 16•11 years ago
|
||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 17•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 1116611
Depends on: 1132539
No longer depends on: 1132539
You need to log in
before you can comment on or make changes to this bug.
Description
•