Closed Bug 1529856 Opened 5 years ago Closed 5 years ago

For the displayed message source of and email using https://outlook.live.com, the text select is malfunctioning.

Categories

(Core :: DOM: Selection, defect, P2)

65 Branch
defect

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox-esr60 --- wontfix
firefox65 --- wontfix
firefox66 - wontfix
firefox67 --- wontfix
firefox68 --- wontfix

People

(Reporter: TezzerFX, Unassigned)

References

()

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:65.0) Gecko/20100101 Firefox/65.0

Steps to reproduce:

Using https://outlook.live.com online email, display an email, under more actions select "View message source". In the message source frame attempt to select a section of text.

Actual results:

From the selection start point (left mouse button held down) all text gets instantly selected up to the end of the message source. Further on right mouse button held down, the copy drop down doesn't appear and the selection is cleared, instead the
page action drop down appears. Double left mouse click can sometimes followed by a selection attempt sporadically work, or may only select whole portions of text, and the copy options are still unavailable!

Expected results:

Normal text selection with copy options should occur in the message source frame as it does for any email or any copy-able html page! This problem seems to have arisen since 65.0.1. Also note: this is not an issue with the last ESR versions of firefox or the last version of Cyberfox.

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:65.0) Gecko/20100101 Firefox/65.0
20190211233335

I haven't tested ESR but in Vivaldi it's not possible to select anything by clicking and dragging. So this may end up moved to Web Compatibility :: Desktop.

Status: UNCONFIRMED → NEW
Has Regression Range: --- → no
Has STR: --- → yes
Component: Untriaged → Selection
Ever confirmed: true
Product: Firefox → Core

I tried versions back to 59; the issue can be reproduced on all the previous versions. HOWEVER, this issue is reproducible on chrome, too. So I think I'd agree with comment 1 that this could be a website issue. This doesn't look a regression to me.

Dear reporter, could you please confirm this is a regression and get the regression window for us by running mozregression tool [1]?

[1] https://github.com/mozilla/mozregression/releases

Flags: needinfo?(TezzerFX)
Keywords: regression

The drag select and the "right mouse button" drop-down copy option menu, work perfectly on the message source, as expected, for
Firefox 52.9.0 32-bit (Vista 64-bit OS) and for Cyberfox 52.9.1 64-bit (W10 1809 OS build 17763.316 64-bit OS) which is based on Firefox 52.9. I think these were last publicly available ESR Firefox versions. So it seems to me this is not explicitly a "website" issue for just any or at least not all browsers. As to Firefox 65.0.1 (64-bit), you tell me!

In order to copy sections of the "message source" to the clipboard, Cyberfox 52.9.1 64-bit has to be used which may have security issues. These Firefox 52.9 versions are the only prior versions available on systems here for testing. Thus it seems this is undoubtedly a Firefox 65.0.1 (64-bit) issue! Unfortunately running the mozregression tool will not be possible, hence as to versions prior to 65.0.1 and after 52.9, someone with the appropriate capabilities will have to perform any regression tests.

[Regression range]:

Last good: build id = 20171215220126
First bad: build id = 20171216220320

Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=4398768baa23a1d1888116a484817e20f2701dcc&tochange=29c2618abb00fdc3358e43902c899f9bd20005b7

Has Regression Range: no → yes

Regression from 2017, I don't think I need to track this for 66.
Maybe we can aim for a fix in 68 or 67.
Hsin-Yi, can you find someone to investigate?

Flags: needinfo?(htsai)

Masayuki,can you please help look at the regression window in comment 4 and suggest which looks suspicious? Thank you.

Flags: needinfo?(masayuki)
Flags: needinfo?(htsai)
Flags: needinfo?(TezzerFX)

Bug 1424633 or bug 1424839 might be related.

Looks like that outlook touches styles and stops propagation of events at focus, blur and mousedown. That might cause canceling selection on Chrome and changing start point of Firefox.

Flags: needinfo?(masayuki)

Thanks Masayuki!
Hi Emilio and Adam, could you please take a look?

Flags: needinfo?(emilio)
Flags: needinfo?(agashlin)

This doesn't seem to be affected by backing out bug 1424839.

Backing out bug 1424633 does seem to fix it, but I'll leave it to Emilio as I don't know this code well.

Flags: needinfo?(agashlin)
Attached patch Backout 1424633 (bad export) (obsolete) — Splinter Review

I'm not suggesting this as a fix, but here's the backout patch that I ended up with, for comparison.

Attachment #9048207 - Attachment description: Backout 1424633 → Backout 1424633 (bad export)
Attachment #9048207 - Attachment is obsolete: true

The page looks like (oversimplified):

<div tabindex=-1 ...>
  <div style="position: fixed">
    ... Contents ...
  </div>
</div>

So before my patch we wouldn't focus the <div tabindex=-1> when clicking inside the contents, but after my patch we do.

Blink does something really weird. If the <div> is empty (apart from out of flow children), then it refuses to make it focusable.

<!doctype html>
<html lang="en">
<div tabindex="-1">
  <div style="position: fixed">
    Fixed
  </div>
</div>

But as long as it has contents, Blink makes it focusable just like us:

<!doctype html>
<html lang="en">
<div tabindex="-1">
  In flow
  <div style="position: fixed">
    Fixed
  </div>
</div>

I think their behavior here is completely inconsistent. Also, WebKit matches us here.

I don't really think aligning with Blink is the right thing to do here... Can we reach out to the Outlook folks rather than doing that?

Flags: needinfo?(emilio)

I'll forward a reference to this to the Mozilla/Microsoft discussion list.

Thanks Emilio and Adam!

Priority: -- → P2

Adam, any updates from the outlook team?

Flags: needinfo?(agashlin)

Since I reported this bug, over the last week something has changed after the 66.0.1 update, namely the drag select and right mouse copy etc. options seem to be fully functioning as expected!

What Happened???

(In reply to Neha Kochar [:neha] from comment #15)

Adam, any updates from the outlook team?

Some Microsoft folks took a look at it and opened a bug on their end March 6, but that was the last I heard of it. I haven't checked personally but from TezzerFX's comment 16 it sounds like it's been fixed.

Flags: needinfo?(agashlin)

(In reply to TezzerFX from comment #16)

Since I reported this bug, over the last week something has changed after the 66.0.1 update, namely the drag select and right mouse copy etc. options seem to be fully functioning as expected!

What Happened???

Maybe Microsoft fixed it then (my best guess).

Closing the bug as resolved per comment 16.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: