Go button does not show when paste text with line break
Categories
(Firefox :: Address Bar, defect)
Tracking
()
People
(Reporter: alice0775, Assigned: daisuke)
References
(Regression)
Details
(Keywords: nightly-community, regression)
Attachments
(1 file)
Step to reproduce:
- Copy the following text
https://bugzilla.mozilla
.org/
- 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
Reporter | ||
Updated•1 year ago
|
Reporter | ||
Updated•1 year ago
|
Comment 1•1 year ago
|
||
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.
Updated•1 year ago
|
Assignee | ||
Comment 2•1 year ago
|
||
Thank you very much for your report, Alice0775!
I also could reproduce this issue.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Comment 3•1 year ago
|
||
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
Comment 5•1 year ago
|
||
bugherder |
Comment 6•1 year ago
|
||
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.
Description
•