Closed Bug 959052 Opened 11 years ago Closed 11 years ago

[NFC] rotating of shrink UI is wrong.

Categories

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::System::Window Mgmt
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(tracking-b2g:backlog)

Status:
RESOLVED FIXED
Milestone:
2.0 S1 (9may)
Project Flags:
tracking-b2g backlog

People

(Reporter: johnhu, Assigned: gasolin)

References

Details

(Whiteboard: [priority][p=1])

Attachments

(3 files)

nfc-rotate-while-shrinking.mp4
(deleted), video/mp4
Details
nfc-rotate-while-shrinking2.mp4
(deleted), video/mp4
Details
pull request redirect to github
(deleted), text/x-github-pull-request
alive
: review+
Details
Attached video nfc-rotate-while-shrinking.mp4 (deleted) —
STR: 1. touch two device 2. rotate the phone <== the shrinking UI shows wrong image. As we discuss with UX offline, the rotation should be disabled in this case.
Blocks: b2g-NFC-2.0
Blocks: NFC-Gaia
No longer blocks: b2g-NFC-2.0
Hi Peter, This is key focus feature in 1.5. Could you put into browser's backlog?
Flags: needinfo?(pdolanjski)
My bad. This one is not for browser.
Flags: needinfo?(pdolanjski)
This issue is caused by we screenshoting the app as the sliding image, and the screenshot won't follow app to do rotation. I just confirm this with John that Alive has said that we should ban the rotation during NFC sharing (which is weird and a edge case) to solve this issue. But I'll confirm this with UX and post the conclusion here.
Hope this bug can be fixed before 5/9.
Target Milestone: --- → 1.5 S1 (9may)
Assignee: nobody → gweng
blocking-b2g: --- → 2.0?
Wesley, Is this NFC bug needed for v2.0
Flags: needinfo?(whuang)
YES, as NFC p2p sharing is key focus in 2.0. ni? to Juwei to decide whether we should ban the rotation for shrinking UI.
Flags: needinfo?(whuang) → needinfo?(jhuang)
Once the two devices paired, the rotation should be DISABLE when screen orientation changed otherwise the display UI would cause error image.
Flags: needinfo?(jhuang)
blocking-b2g: 2.0? → backlog
Whiteboard: [priority]
This bug blocks NFC 2.0 release.
Blocks: b2g-NFC-2.0
take it
Assignee: gweng → gasolin
Attached file pull request redirect to github (deleted) —
tested on real device
Attachment #8419231 - Flags: review?(alive)
Comment on attachment 8419231 [details] pull request redirect to github commented
Attachment #8419231 - Flags: review?(alive)
Comment on attachment 8419231 [details] pull request redirect to github revised and tested in browser(orient:no default) and music(orient:default) app. after shrink-stop music app's orientation won't be unlocked.
Attachment #8419231 - Flags: review?(alive)
Attachment #8419231 - Flags: review?(alive) → review+
merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/c859fc43c23713844239a13172ca7e6b0c94cbcb thanks!
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [priority] → [priority][p=1]
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: