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)

x86
macOS
defect
Not set
normal

Tracking

(tracking-b2g:backlog)

RESOLVED FIXED
2.0 S1 (9may)
tracking-b2g backlog

People

(Reporter: johnhu, Assigned: gasolin)

References

Details

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

Attachments

(3 files)

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.
Attached video nfc-rotate-while-shrinking2.mp4 (deleted) —
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 → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: