Closed Bug 1430672 Opened 2 years ago Closed 2 years ago
Modifying search field moves cursor to the end
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
Fixed in bug 1432842.
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 :)
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.
If you're worried about risk, let's just let it ride the 60 train.
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
Cool - i'll try and give this a try next week. Thanks
QA Whiteboard: [good first verify]
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]
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.
You need to log in before you can comment on or make changes to this bug.