[webkit-reviews] review denied: [Bug 199413] Web Inspector: Elements: DOM breakpoints context menu items inserted first when accessed via status image : [Attachment 373342] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 2 11:43:23 PDT 2019


Devin Rousso <drousso at apple.com> has denied Matt Baker <mattbaker at apple.com>'s
request for review:
Bug 199413: Web Inspector: Elements: DOM breakpoints context menu items
inserted first when accessed via status image
https://bugs.webkit.org/show_bug.cgi?id=199413

Attachment 373342: Patch

https://bugs.webkit.org/attachment.cgi?id=373342&action=review




--- Comment #2 from Devin Rousso <drousso at apple.com> ---
Comment on attachment 373342
  --> https://bugs.webkit.org/attachment.cgi?id=373342
Patch

r-, I did this intentionally.  If a user shows a context menu specifically for
the breakpoint icon, it likely means that they want to do something with
breakpoints, as it's pretty "small" and is way far over in the gutter area.  As
such, we should put the actions for breakpoints closer to the top.


More information about the webkit-reviews mailing list