Closed Bug 1022096 Opened 11 years ago Closed 11 years ago

[Messages] sms text truncated in delete messages mode

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::SMS
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:2.0+, b2g-v2.0 verified, b2g-v2.1 verified)

Status:
VERIFIED FIXED
Milestone:
2.0 S4 (20june)
Project Flags:
blocking-b2g 2.0+
Tracking Flags:
Tracking Status
b2g-v2.0 --- verified
b2g-v2.1 --- verified

People

(Reporter: ericcc, Assigned: julienw)

References

Details

(Keywords: regression, Whiteboard: [2.0-FL-bug-bash][not-part-of-initial-sprint])

User Story

Build Information

Device: 
Gaia      
Gecko     
BuildID   
Version   
ro.build.version.incremental=
ro.build.date=

Description

Steps to Reproduce

Expected Results

Actual Results

Other Notes

Reproduction Frequency:

Attachments

(2 files)

Delete Messages.png
37.90 KB, image/png
Details
github PR
46 bytes, text/x-github-pull-request
azasypkin
: review+
Details | Review
Attached image Delete Messages.png —
### Build Information Device: Flame Gaia 908f94fda04462001ece86e6b6c15ad8b05f7526 Gecko https://hg.mozilla.org/mozilla-central/rev/5e8e9d864a4d BuildID 20140605160202 Version 32.0a1 ro.build.version.incremental=94 ro.build.date=Tue May 20 09:29:20 CST 2014 ### STR 1. Prepare an sms longer than a line 2. Open Messages -> Option -> Delete Messages 3. Characters cut off in the middle.. ### Reproduction Frequency: 100%
See Also: → 1022118
QA wanted: does it reproduce on v1.4 or v1.3? (see attachment 8436293 [details])
Keywords: qawanted
The issue DOES reproduce on the 1.4 Flame. The issue does NOT reproduce on the 1.3 Buri. 1.4 Environmental Variables: Device: Flame 1.4 Build ID: 20140609000201 Gaia: 8b239e41bbd85aa7b6a2c5d388e775ba7de6fb2b Gecko: e3f85877db29 Version: 30.0 (1.4) Firmware Version: v10G-2 1.3 Environmental Variables: Device: Buri 1.3 Build ID: 20140607024002 Gaia: e049de79ff46fcab51f29a88d5869c5efb48fc3c Gecko: 90e7f697961a Version: 28.0 (1.3) MOZ Firmware Version: v1.2device.cfg
Keywords: qawanted
QA Contact: bmorgan
What about 1.3 Flame?
Keywords: qawanted
This issue does NOT occur on the Flame 1.3 Base Image: Environmental Variables: Device: Flame 1.3 Build ID: 20140520094859 Gaia: a73235d23685e9898f40647cebd83b3fcbfd0117 Gecko: b637b0677e15318dcce703f0358b397e09b018af Version: 28.0 (1.3) Firmware Version: v10G-2
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(jmitchell)
Keywords: qawanted
QA Contact: bmorgan → ychung
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell)
Joshua - The triage marking here looks incorrect. The tester here did not mark the bug with the regression & regressionwindow-wanted keyword. If the bug reproduces on 1.4, but doesn't reproduce on 1.3, then it's a regression.
Flags: needinfo?(jmitchell)
Right you are, sorry about that!
Flags: needinfo?(jmitchell)
Keywords: regression, regressionwindow-wanted
Minor regression.
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Keywords: regressionwindow-wanted
The regression would likely point one of the visual refresh bugs anyway, so it would not be really useful.
blocking-b2g: --- → 2.0?
blocking-b2g: 2.0? → 2.0+
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [QAnalyst-Triage+]
Keywords: regressionwindow-wanted
QA: can we test the same STR on other apps that use the same "edit mode" panel? (eg: Contacts with a contact that has a long name, or maybe email) Removing regressionwindow-wanted because it won't give us anything useful (will likely point to the edit mode visual refresh).
Keywords: regression, regressionwindow-wanted → qawanted
Keywords: regression
Assignee: nobody → felash
Attached file github PR —
Hey Oleg, do you have time to review this simple patch? It used to work by chance in v1.4 because the text was (incorrectly) not displayed on the right of the panel.
Attachment #8440796 - Flags: review?(azasypkin)
Comment on attachment 8440796 [details] [review] github PR (In reply to Julien Wajsberg [:julienw] from comment #10) > Created attachment 8440796 [details] [review] > github PR > > Hey Oleg, > > do you have time to review this simple patch? > > It used to work by chance in v1.4 because the text was (incorrectly) not > displayed on the right of the panel. Sure! Looks fine to me. Left small question at github.
Attachment #8440796 - Flags: review?(azasypkin) → review+
master: 68c0d6a6bc95206f179dab5b26d1a615927006c3
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
status-b2g-v2.0: --- → affected
status-b2g-v2.1: --- → fixed
Target Milestone: --- → 2.0 S4 (20june)
Issue has been marked resolved - removing QA-Wanted keywords
Keywords: qawanted
Blocks: sms-sprint-3
Whiteboard: [2.0-FL-bug-bash] → [2.0-FL-bug-bash][not-part-of-initial-sprint]
Works okay with these 2 builds. Flame Aurora v2.0 Gaia 1dd043857399c713e3b509c0ed31bdf20326f08b Gecko https://hg.mozilla.org/releases/mozilla-aurora/rev/3f9d7a3a0b7b BuildID 20140707160206 Version 32.0a2 ro.build.version.incremental=109 ro.build.date=Mon Jun 16 16:51:29 CST 2014 B1TC00011220 Flame Master v2.1 Gaia e935f4ff190b76c70d9b2af8856c542a6e4a7546 Gecko https://hg.mozilla.org/releases/mozilla-aurora/rev/3f9d7a3a0b7b BuildID 20140707160206 Version 32.0a2 ro.build.version.incremental=109 ro.build.date=Mon Jun 16 16:51:29 CST 2014 B1TC00011220
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: