[Webkit-unassigned] [Bug 138991] Web Inspector: Pause Reason Improvements (Breakpoint, Debugger Statement, Pause on Next Statement)

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=138991

--- Comment #2 from Joseph Pecoraro <joepeck at webkit.org> ---
Created attachment 242110
  --> https://bugs.webkit.org/attachment.cgi?id=242110&action=review
[IMAGE] Images

These images are what I have in this patch. This is certainly NOT final UI, it was mostly to make sure functionality was working as expected.

See also the screenshots on bug 63096.

I discussed the UI a bit with others, here are some thoughts being thrown around.

  1. Keep the section title name static, "Pause Reason".
    - easier to explain the UI to someone.
    - existing Assertion/Exception/CSP Violation titles could move into the section
    - for exceptions, that could also explain that we are showing the thrown object description
  2. Goto arrow could get confused with Continue button.
  3. We could show the location in more places.
    - the goto arrow tooltip shows it, but it maybe be nice to know at a glance

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20141122/803962cc/attachment-0002.html>