RESOLVED FIXED 167651
Web Inspector: User Settings: Secondary-clicking the New Tab and Settings tabs should not produce a context menu
https://bugs.webkit.org/show_bug.cgi?id=167651
Summary Web Inspector: User Settings: Secondary-clicking the New Tab and Settings tab...
Blaze Burg
Reported 2017-01-31 10:18:55 PST
Via Nelson Thachuk: * SUMMARY - Secondary-clicking the New Tab and Settings tabs should not produce a context menu * STEPS TO REPRODUCE 1. Launch Safari 2. Navigate to a webpage 3. Show Web Inspector (⌥⌘I) 4. Secondary-click the New Tab and Settings buttons * EXPECTED RESULTS - Nothing should occur * ACTUAL RESULTS - Context menu with “Copy Image” that doesn’t copy anything to the clipboard
Attachments
Patch (5.54 KB, patch)
2017-01-31 11:08 PST, Devin Rousso
joepeck: review+
Patch (5.22 KB, patch)
2017-01-31 12:28 PST, Devin Rousso
no flags
Blaze Burg
Comment 1 2017-01-31 10:19:19 PST
Devin Rousso
Comment 2 2017-01-31 11:08:03 PST
Joseph Pecoraro
Comment 3 2017-01-31 11:13:27 PST
Comment on attachment 300237 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=300237&action=review > Source/WebInspectorUI/UserInterface/Views/TabBar.js:376 > + get newTabTabBarItem() { return this._newTabTabBarItem; } Getters like these should be at the top of the Public section.
Joseph Pecoraro
Comment 4 2017-01-31 11:15:47 PST
Comment on attachment 300237 [details] Patch r=me
Devin Rousso
Comment 5 2017-01-31 12:28:29 PST
WebKit Commit Bot
Comment 6 2017-01-31 13:05:52 PST
Comment on attachment 300246 [details] Patch Clearing flags on attachment: 300246 Committed r211445: <http://trac.webkit.org/changeset/211445>
WebKit Commit Bot
Comment 7 2017-01-31 13:05:55 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.