[Webkit-unassigned] [Bug 27514] add support for watched expressions

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 4 08:16:57 PDT 2009


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


Patrick Mueller <pmuellr at yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #39057|                            |review?
               Flag|                            |
  Attachment #34170|0                           |1
        is obsolete|                            |




--- Comment #7 from Patrick Mueller <pmuellr at yahoo.com>  2009-09-04 08:16:56 PDT ---
Created an attachment (id=39057)
 --> (https://bugs.webkit.org/attachment.cgi?id=39057)
proposed patch - 2009/09/04

final for now, definitely working version

Couple things to clean up, which I'd like to do later, after getting the meat
of the code in:

- I suspect folks won't like the red delete icon that shows up in the right
column; I don't, but couldn't find anything better.  Suggestions?  I was
thinking maybe taking the existing icon I'm using, but convert it to grayscale.

- not really happy with the Add and Refresh buttons.  Too big.  I tried a
smaller font at one point, one pt smaller, but that seemed too small.  Was
wondering about making these link-like instead of button-like.

- there are some changes to existing code elsewhere that should be made;
pfeldman suggested some proposed changes to make the Object.describe() vs
proxy.description stuff simpler, making the console.eval code "public" (maybe
we should move it somewhere else since it's not console specific any more), and
property sorting should move to utilities.  I'd like to do these later.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list