Closed Bug 970028 Opened 11 years ago Closed 11 years ago

[Sora][Message]The screen shaked when back to message edit interface.

Categories

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

defect

Tracking

(tracking-b2g:backlog, b2g-v1.4 verified, b2g-v2.0 verified)

VERIFIED FIXED
1.4 S4 (28mar)
tracking-b2g backlog
Tracking Status
b2g-v1.4 --- verified
b2g-v2.0 --- verified

People

(Reporter: sync-1, Assigned: alive)

References

Details

(Whiteboard: [in-bubble-tea])

Attachments

(3 files, 1 obsolete file)

Firefox OS v1.3 Mozilla build ID: 20140105004001 DEFECT DESCRIPTION: ->SMS screen shake. REPRODUCING PROCEDURES: ->Enter"Messages" to create a message; ->Tap"Setting"icon to enter message settings list; ->Tap"x" back to the message edit interface; EXPECTED BEHAVIOUR: ->Should display normally. ASSOCIATE SPECIFICATION: TEST PLAN REFERENCE: TOOLS AND PLATFORMS USED: USER IMPACT: REPRODUCING RATE:100% For FT PR, Please list reference mobile's behavior:
Can you please share a video? thanks!
Flags: needinfo?(sync-1)
Attached video VID_0020.3gp (obsolete) (deleted) —
I upload a video about the shack of transform from sms to message settings, and then back to sms.
Attached video VID_0021_1.3gp (deleted) —
Please use this one to check, and ignore last one.
Attachment #8373195 - Attachment is obsolete: true
oki, looks like an activity window management issue Thanks !
Component: Gaia::SMS → Gaia::System::Window Mgmt
Flags: needinfo?(sync-1) → needinfo?(alive)
Assignee: nobody → alive
Flags: needinfo?(alive)
blocking-b2g: --- → 1.3?
Adding qawanted to check if QA is able to reproducible on Mozilla's side ..I tried on a buri device on the build id : 20140213175711 but was unable to hit the issue as described... Or may be the reporter can try this once again on the latest build to confirm the issue.
Flags: needinfo?(sync-1)
Keywords: qawanted
This is too minor to invest QA resources into - clearing the flag.
Keywords: qawanted
Noemi Can you please prioritize this within TEF? Moz cannot test this in-house, please check if this is an issue.
Flags: needinfo?(noef)
(In reply to Preeti Raghunath(:Preeti) from comment #7) > Noemi > > Can you please prioritize this within TEF? Moz cannot test this in-house, > please check if this is an issue. No, that's not the reason why. The priority of the bug is too low, so we should not invest QA resources here. Moving to backlog.
blocking-b2g: 1.3? → backlog
Flags: needinfo?(noef)
This pr can still be reproduced on version with Mozilla build ID: 20140208004002.
Blocks: 949912
Attached file Patch for bubble-tea (deleted) —
The patch is based on bug 961800.
Attachment #8381987 - Flags: review?(timdream)
Attachment #8381987 - Flags: review?(timdream) → review+
Depends on: 961800
Flags: needinfo?(sync-1)
Whiteboard: [in-bubble-tea]
Alive, I don't think we need 1.4+ to land the patch on master, if it does not depend on other bubble-tea work. Let's just land it :)
Sorry, we need to add some css styles here. I am wrong in my previous response. :/
Anyway I will make a v1.4 patch!
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reverted in https://github.com/mozilla-b2g/gaia/commit/0ef85e9bafc4e3ee1989f1c0e455f82d8313d428 because something in this or one of the six other patches that the b2g bumper bot included in a single push to b2g-inbound broke gaia-ui tests on TBPL: bumper bot's push's contents: https://hg.mozilla.org/integration/b2g-inbound/rev/0c2938cd910b The gaia-ui failure: https://tbpl.mozilla.org/php/getParsedLog.php?id=36421229&tree=B2g-Inbound In the future, can we please not merge seven patches in the course of 5 minutes? It would make tracking down regressions much simpler if they were spaced several minutes apart each.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Alive, please land this to master with bug 961800 yourself without considering bubble-tea, thanks.
Flags: needinfo?(alive)
Got
Flags: needinfo?(alive)
Attached file Patch for master (deleted) —
Cherry-pick bubble-tea and land part of css rules from bug 961800. Awaiting travis green.
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Should we ask for 1.4 approval on this?
Flags: needinfo?(alive)
Comment on attachment 8395536 [details] Patch for master NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] [Bug caused by] (feature/regressing bug #): [User impact] if declined: [Testing completed]: [Risk to taking this patch] (and alternatives if risky): No risk because it's nearly only css change. Honored new app transition controller gave us the ability to change the opening/closing transition easily as we like. [String changes made]:
Attachment #8395536 - Flags: approval-gaia-v1.4?
Flags: needinfo?(alive)
Comment on attachment 8395536 [details] Patch for master Alive, request to please help fill all approval request comments inline an any approval request next time. Have also built these guilelines recently which may help : https://wiki.mozilla.org/Release_Management/Uplift_rules#Guidelines_on_approval_comments For now given the risk of the patch and the attached tests this looks good, hence approving. Requesting QA verification in parallel
Attachment #8395536 - Flags: approval-gaia-v1.4? → approval-gaia-v1.4+
Depends on: 990802
1.4: Smooth transition between screens when backing out from Message Settings while creating message or at Messages home screen. 2.0: Smooth transition between screens when backing out from Message Settings to Messages home screen. Will not allow access to settings when creating a message. 1.4 Environmental Variables: Device: Buri 1.4 MOZ BuildID: 20140527000202 Gaia: 0542778892a294d224e75af4a76be5d42938bc90 Gecko: d583ae109f54 Version: 30.0 Firmware Version: v1.2-device.cfg 2.0 Environmental Variables: Device: Flame 2.0 MOZ BuildID: 20140527040202 Gaia: 6a391274cd436f8f0d1fad2db8c6b4805703259c Gecko: cbe4f69c2e9c Version: 32.0a1 Firmware Version: 10g-2
Status: RESOLVED → VERIFIED
Keywords: verifyme
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: