Closed Bug 1160261 Opened 5 years ago Closed 3 years ago

[SMS][Text Selection] Copy & Cutting a block of text that includes an attachment will remove the attachment but Pasting that block back will not restore it

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(tracking-b2g:backlog, b2g-v2.2 affected, b2g-master affected)

RESOLVED WONTFIX
tracking-b2g backlog
Tracking Status
b2g-v2.2 --- affected
b2g-master --- affected

People

(Reporter: jmitchell, Unassigned)

References

()

Details

(Whiteboard: [3.0-Daily-Testing])

Attachments

(1 file)

Description:
In SMS, if you add an attachment that attachment can become affected by text selection. If you 'select all' it will become part of the selection and will be deleted if you select 'cut', however, it will not be restored upon a paste function. It seems like it might be best if attachments were not affected by text selection functionality. 


Repro Steps:
1) Update a Flame to 20150430010201
2) Launch SMS and compose a new message
3) Type some text into the message field
4) Add an attachment 
5) Long press on the text in the message field and select 'select all'
6) select 'Cut', then 'paste' the text back in

Actual:
Attachments are affected by text selection options


Expected:
Attachments will not be affected by text selection options
-or-
if the attachment can be removed with the text selection options one might expect it could also be restored with them



Environmental Variables:
Device: Flame 3.0
Build ID: 20150430010201
Gaia: db8ea705c0fd1b1684807f5a8e837bb9a36a6f96
Gecko: 4b9b12c248dc
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 40.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0




Repro frequency: 6/6
See attached: logcat, video clip:http://youtu.be/SdIFkriaT_8
This issue also reproduces in Flame KK 2.2; text selection is not supported prior to that

Device: Flame 2.2 (KK - Nightly - Full Flash - 319mem)
Build ID: 20150429002501
Gaia: 1b7aa7e60788668ed09abf76022dfa231dbe88d4
Gecko: d38ff4717f39
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage?][COM=Text Selection]
Flags: needinfo?(pbylenga)
NI on component owner for nomination decision and assignment.
QA Whiteboard: [QAnalyst-Triage?][COM=Text Selection] → [QAnalyst-Triage+][COM=Text Selection]
Flags: needinfo?(pbylenga) → needinfo?(gchang)
I don't know what is the correct behavior here. We don't support anything than text copy/paste in v2.2.
Hey Steve, can you find out what is the correct behavior here? Also see what the behavior in Email is?
Flags: needinfo?(schung)
If the mail is in html mode, text selection is disabled; if the mail is text mode, attachment and text is separated. So this behavior does not exist in email.

I just discussed with Omega and we don't think there would be any perfect solution until platform support non-text copy/paste. Hi Peter, do you know any existing gecko bug related to this requirement? Thanks!
Flags: needinfo?(schung) → needinfo?(pchang)
Flags: needinfo?(gchang)
(In reply to Steve Chung [:steveck] from comment #5)
> If the mail is in html mode, text selection is disabled; if the mail is text
> mode, attachment and text is separated. So this behavior does not exist in
> email.
> 
> I just discussed with Omega and we don't think there would be any perfect
> solution until platform support non-text copy/paste. Hi Peter, do you know
> any existing gecko bug related to this requirement? Thanks!

I assume you are talking about the support of image paste functionality, please correct me if I'm wrong. Before we had discussed with UX about the same issue during clipboard API discussion, the UX behavior is not very clear for FxOS because not every app supports the image paste option, for example, you can't copy the attachment from message to line. And there are lots of different corner cases that we need to handle for the image paste option.
Flags: needinfo?(pchang)
No longer blocks: AccessibleCaret
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.