1609918 - <New Line> at Google Sheets

Status: RESOLVED DUPLICATE of bug 1605018

(Core :: DOM: Editor, defect)

Description

[Ivan Ivanovich Ivanov]

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

Steps to reproduce:

1. Open https://https://docs.google.com/spreadsheets/u/0/create?usp=sheets_home&ths=true

2. Write any letter in any cell, for example, the letter "q" and press Enter.

3. Return to the previous cell by double-clicking the mouse and erase the letter “q” with the Backspace key, then press Enter again.

Actual results:

A new line character will remain in an empty cell (it can be found by downloading the file and pressing Ctrl+J in the MS Excel search window)

Expected results:

The cell should not contain anything, not even invisible characters (This problem is not observed in TOR Browser \ Firefox 68.4.1 ESR x64)

Comment 1

[Raluca Popovici, Desktop QA]

Attachment: 2020-01-21_14h25_33.png

Comment 2

[Raluca Popovici, Desktop QA]

Comment 3

[Raluca Popovici, Desktop QA]

Hi,
I can reproduce this issue on Firefox 73.0b7 (2020-01-18) , Firefox 72.0.2(2020-01-17), Firefox 74.0a1(2020-01-21).

Comment 4

[Edgar Chen [:edgar]]

(In reply to vip818 from comment #0)

(This problem is not observed in TOR Browser \ Firefox 68.4.1 ESR x64)

This looks like a regression.

Comment 5

[Alice0775 White]

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=a351f679051364c93fd7b014960537a45599c6dc&tochange=483c129b10e3d94850e3a6cc94545e11139a6343

Comment 6

[Alice0775 White]

If spoofed UA, the problem is gone.
i.e., After logged in google with default UA. And then change UA to "Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4021.2 Safari/537.36", then open the spreadsheets.

So, I think this seems the site problem.

Comment 7

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Thank you, Alice-san!

Indeed, from my environment, 3784985188-waffle_js_rod_core__ja.js refers type attribute of <br> element to check whether it's value is _moz or not.

function Mrc(a, b) {
  if (3 == a.nodeType) {
    var c = a.nodeValue;
    'trix-quote' == a.parentNode.className && (c = c.replace(/\u02BA/g, '"'));
    c = c.replace(/\uFEFF/g, '');
    0 < c.length && b.push(c)
  } else if ('BR' == a.nodeName)
    pf && '_moz' == a.getAttribute('type') || b.push('\n');
  else if (!(pf && 'DIV' == a.nodeName && 1 == a.childElementCount && 'BR' == a.firstChild.nodeName || 'SPAN' == a.nodeName && ax(a, 'jfk-tooltip-data') || 'STYLE' == a.nodeName))
    for (a = a.firstChild; a; )
      Mrc(a, b), a = a.nextSibling;
  return b
}

Mike, could you contact Google?

Comment 9

[Mike Taylor [:miketaylr]]

Will post updates in 1605018, thanks for the diagnosis Masayuki-san!