[Webkit-unassigned] [Bug 53359] Web Inspector: Allow the console to persist on page refresh or navigation

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 9 11:26:51 PDT 2011


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





--- Comment #8 from Pavel Feldman <pfeldman at chromium.org>  2011-06-09 11:26:51 PST ---
> To be clear, my current patch doesn't let you expand dead objects either, so it _appears_ to have no functional difference to clearing the backend objects, but then again, I'm totally new to this part of the code, and I don't know the logic involved.

We do, and we are suggesting that you clear the list!

> I can certainly implement this, but I'd strongly encourage you to consider a more uniform UI like in the current patch. It seems confusing to have two different ways of achieving the same functionality in two panels (especially if one is a button and the other tucked away in a context menu).

Console can be accessed not only as a panel, but as a drawer on any other panel as well (via pressing Esc). Preserving log does make sense both for Network panel and Console, but it does not make sense for the rest of the panels. So putting it everywhere will be confusing, especially on the Timeline, Profiler and Audit panels.

> If the icon is the only thing that's confusing, I can try my hand at adding something different. If it's also a matter of other panels not respecting the setting (yet), that should give us more impetus to add similar functionality to those panels as soon as possible. I think it would be fantastic to have all panels implement "preserve log" functionality.

I disagree here, I don't see how Elements or Resources can possibly "preserve log". The information they expose is state-driven, not log-driven. So no, the icon is not the only concern. My advice for you was that if you want to have this change landed in a foreseeable future, you should minimize the surface of the UI changes. Context menu seems like a good fit.

-- 
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