Summary: | Web Inspector: delete-by-word and similar shortcuts should add text to the WebCore kill ring | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | BJ Burg <bburg> | ||||
Component: | Web Inspector | Assignee: | BJ Burg <bburg> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bburg, commit-queue, darin, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=151663 | ||||||
Bug Depends on: | 151157 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
BJ Burg
2015-11-16 10:39:06 PST
Created attachment 266030 [details]
Proposed Fix
Comment on attachment 266030 [details] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=266030&action=review > Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorTextKillController.js:44 > + // FIXME: these keybindings match CodeMirror's default keymap for OS X. > + // They should probably be altered for Windows / Linux someday. Unfortunate to have to hard code these like this. Generally speaking, Safari and WebKit respect the OS X system-wide key bindings <https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/EventOverview/TextDefaultsBindings/TextDefaultsBindings.html>. Comment on attachment 266030 [details] Proposed Fix Clearing flags on attachment: 266030 Committed r192804: <http://trac.webkit.org/changeset/192804> All reviewed patches have been landed. Closing bug. (In reply to comment #4) > Comment on attachment 266030 [details] > Proposed Fix > > View in context: > https://bugs.webkit.org/attachment.cgi?id=266030&action=review > > > Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorTextKillController.js:44 > > + // FIXME: these keybindings match CodeMirror's default keymap for OS X. > > + // They should probably be altered for Windows / Linux someday. > > Unfortunate to have to hard code these like this. Generally speaking, Safari > and WebKit respect the OS X system-wide key bindings > <https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/ > EventOverview/TextDefaultsBindings/TextDefaultsBindings.html>. Wow, I had no idea that this exists! I'll file a bug to see if we can integrate it. |