783498 - [rule view] Can't copy text from the editor

Status: RESOLVED DUPLICATE of bug 788458

(DevTools :: Inspector, defect)

Description

[Paul Rouget [:paul]]

When the editor is open, selecting text in it and then doing ctrl-c doesn't copy the right text.

Comment 1

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Attachment: Fix

This patch fixes this issue but the context menu still does not have "copy selection" enabled. This is because of a browser bug that fails to detect whether text is selected in all situations (in our case within our editor).

Unfortunately, logging a new bug ... or rather, creating a reproduce-able test case would be very difficult.

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 658136 [details] [diff] [review]
Fix

After some discussion we have decided to always allow allow copy from the context menu.

Comment 3

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 658136 [details] [diff] [review]
Fix

Because we cannot get the selection from the input field we cannot copy the text using the context menu (we would need to know where the range starts and ends).

Seems like this is the best we can do.

Comment 4

[Paul Rouget [:paul]]

Comment on attachment 658136 [details] [diff] [review]
Fix

Not sure if I'm supposed to review attachment 658136 [details] [diff] [review] after the discussion we had. 

Anyway:

> Because we cannot get the selection from the input field

Can you file a bug please?

This patch doesn't seem to work for me. Also, I still see the greyed menuitem.

Comment 5

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Fixed in bug 788458