Closed Bug 1054635 Opened 10 years ago Closed 10 years ago

[Messages] Context menu/long tap on container area in Composer panel causes javascript error

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
minor

Tracking

(b2g-v2.2 fixed)

RESOLVED FIXED
2.2 S1 (5dec)
Tracking Status
b2g-v2.2 --- fixed

People

(Reporter: azasypkin, Assigned: rishav_, Mentored)

Details

(Whiteboard: [sms-papercuts][lang=js][good first bug])

Attachments

(1 file)

Context menu/long tap on container area in Composer panel causes javascript error

[JavaScript Error: "TypeError: messageBubble is null" {file: app://sms.gaiamobile.org/js/thread_ui.js" line: 1983}]

Haven't noticed any user facing issues though.
Would you be willing to mentor a contributor through this?
Flags: needinfo?(azasypkin)
Mentor: azasypkin
Whiteboard: [sms-papercuts] → [sms-papercuts][lang=js][good first bug]
Flags: needinfo?(azasypkin)
Assignee: nobody → rishav006
Status: NEW → ASSIGNED
Hi Juliew
As Oleg is on PTO, i asked review from you. Here is the patch. Please have a look :)
Thanks
Attachment #8507887 - Flags: review?(felash)
Comment on attachment 8507887 [details] [review]
Bug 1054635 - Context menu/long tap on container area in Composer panel causes javascript error

Please add a unit test :)

thanks
Attachment #8507887 - Flags: review?(felash)
Comment on attachment 8507887 [details] [review]
Bug 1054635 - Context menu/long tap on container area in Composer panel causes javascript error

Hi Julien
Have a look on PR.
Hope it's fine now.
Thanks
Attachment #8507887 - Flags: review?(felash)
Comment on attachment 8507887 [details] [review]
Bug 1054635 - Context menu/long tap on container area in Composer panel causes javascript error

r=me
thanks :)
Attachment #8507887 - Flags: review?(felash) → review+
https://github.com/mozilla-b2g/gaia/commit/9c963e394917bfe49925e403e522fd763d15a306
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S1 (5dec)
Summary: Context menu/long tap on container area in Composer panel causes javascript error → [Messages] Context menu/long tap on container area in Composer panel causes javascript error
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: