Closed
Bug 1430672
Opened 7 years ago
Closed 7 years ago
Modifying search field moves cursor to the end
Categories
(DevTools :: Debugger, defect)
DevTools
Debugger
Tracking
(firefox-esr52 unaffected, firefox58 wontfix, firefox59 wontfix, firefox60 verified, firefox61 verified)
VERIFIED
FIXED
Firefox 60
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox58 | --- | wontfix |
firefox59 | --- | wontfix |
firefox60 | --- | verified |
firefox61 | --- | verified |
People
(Reporter: Oriol, Assigned: jlast)
References
Details
(Keywords: regression)
1. Go to https://example.com
2. Open the debugger
3. Press Ctrl+Shift+F
4. Enter 'abc'
5. Press Home key
6. Press Del key (forward delete)
Result: 'bc<cursor>'
Expected: '<cursor>bc'
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=2f49d6e7e883cb91a6d524ba43725e59243ba208&tochange=129dfaaf265a5eb9b18681fded2c3910167ddcff
Reporter | ||
Comment 1•7 years ago
|
||
Reporter | ||
Comment 2•7 years ago
|
||
Fixed in bug 1432842.
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → affected
status-firefox59:
--- → affected
status-firefox60:
--- → fixed
status-firefox-esr52:
--- → unaffected
Depends on: 1432842
Resolution: --- → FIXED
Comment 3•7 years ago
|
||
Is it worth trying to backport the fix for this issue (by itself) to Beta for the benefit of our DevEdition audience? If so, please attach a patch and request approval :)
Assignee: nobody → jlaster
Flags: needinfo?(jlaster)
Target Milestone: --- → Firefox 60
Assignee | ||
Comment 4•7 years ago
|
||
Good question :RyanVM. My guess is that it is fairly low impact and the patch could be a bit tricky. I'm leaning towards no, but could be persuaded.
Flags: needinfo?(jlaster)
Comment 5•7 years ago
|
||
If you're worried about risk, let's just let it ride the 60 train.
Reporter | ||
Comment 6•7 years ago
|
||
I don't think it's tricky. Remove few lines from SearchInput.js should suffice
https://github.com/devtools-html/debugger.html/commit/282c688f9a0150aab6b810a8d57a874a5c005e0a#diff-5f2aad7d5f0db62d1bdb6c5f0b64e714
Assignee | ||
Comment 7•7 years ago
|
||
Cool - i'll try and give this a try next week. Thanks
Updated•7 years ago
|
QA Whiteboard: [good first verify]
Comment 8•7 years ago
|
||
I have reproduced this bug with Nightly 59.0a1 (2018-01-15) on Windows 10, 64 Bit!
This bug's fix is verified with latest Beta!
Bulid ID - 20180419200216
User Agent - Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:60.0) Gecko/20100101 Firefox/60.0
QA Whiteboard: [good first verify] → [good first verify] [testday-20180420]
Comment 9•7 years ago
|
||
Reproduced the bug on Nightly 59.0a1 (2018-01-07) under Windows 10 x64.
Bug fix is verified with: Firefox 60.0b15(64 bit) and latest Nightly 61.0a1(2018-04-25).
Tests were performed under Windows 10x64, MacOS X 10.10.5 and Ubuntu 16.04x64.
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•