Closed Bug 1826184 Opened 1 year ago Closed 1 year ago

Go button does not show when paste text with line break

Categories

(Firefox :: Address Bar, defect)

Firefox 88
Desktop
Windows 10
defect

Tracking

()

VERIFIED FIXED
113 Branch
Tracking Status
firefox-esr102 --- wontfix
firefox111 --- wontfix
firefox112 --- wontfix
firefox113 --- verified

People

(Reporter: alice0775, Assigned: daisuke)

References

(Regression)

Details

(Keywords: nightly-community, regression)

Attachments

(1 file)

Step to reproduce:

  1. Copy the following text
https://bugzilla.mozilla
.org/
  1. Paste to address bar

Actual results:
Go button does not show.

Expected results:
Go button should show.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=194e31587e6c4174702a223b448e8748b1b4a144&tochange=7e7982e97ed002b61e64b556f85866b5f6b9ce46

Suspect: Bug 1327589

Version: Trunk → Firefox 88
Summary: Go button does not show when paste text → Go button does not show when paste text with line break

Set release status flags based on info from the regressing bug 1327589

:daisuke, since you are the author of the regressor, bug 1327589, could you take a look?

For more information, please visit auto_nag documentation.

Flags: needinfo?(daisuke)

Thank you very much for your report, Alice0775!
I also could reproduce this issue.

Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Flags: needinfo?(daisuke)

The direct reason why the go-button was hidden when pasting text is that
"usertyping" attribute will not be true.
https://searchfox.org/mozilla-central/rev/8433b62e54fd30663e82f090c4d31554531a2e66/browser/base/content/browser.css#754
When pasting data and removing white space or line break, call preventDefault().
https://searchfox.org/mozilla-central/rev/8433b62e54fd30663e82f090c4d31554531a2e66/browser/components/urlbar/UrlbarInput.sys.mjs#3504-3505
The other hand, if not removing any chars, _on_input() will be called because
don't call preventDefault(). And, the attribute is set in _on_input().
https://searchfox.org/mozilla-central/rev/8433b62e54fd30663e82f090c4d31554531a2e66/browser/components/urlbar/UrlbarInput.sys.mjs#3343

So, we set the attribute even if pasting data is changed as same as _on_input().

Pushed by dakatsuka.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4ccbf0ef137f
Show go-button even if pasting multiline text on urlbar r=adw
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch

Reproducible on a 2023-04-03 Nightly build on Windows 10.
Verified as fixed on Nightly 113.0a1(build ID: 20230409095525) on Windows 10, Ubuntu 22, macOS 12.
Go button is now visible and works properly.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: