Bug 34319 – Web Inspector: wire together buttons on welcome view and on status bar

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Mikhail Naganov]

In Profiles panel, pressing buttons on the Welcome view doesn't change their look, only status bar button repaints itself. This is confusing, because it may appear to an user that the welcome view button doesn't work.

I think, welcome view and status bar buttons must update their look regardless of what button was pressed.

[Mikhail Naganov]

Created attachment 47692 [details]
proposed patch

[Pavel Feldman]

Comment on attachment 47692 [details]
proposed patch

It is now even more confusing - once you toggle a button, welcome message stops making sense: it suggests that red button starts the profiler.

[Mikhail Naganov]

Created attachment 51745 [details]
another approach

OK, another approach, this time by limiting functionality.

This is how it looks like: http://i42.tinypic.com/dgt5wk.png

[Pavel Feldman]

Comment on attachment 51745 [details]
another approach

Note that you could attach the screenshot to the bug.

[Mikhail Naganov]

Manually committed r56711: http://trac.webkit.org/changeset/56711

Author: [e-mail] <[e-mail]@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Mon Mar 29 08:52:19 2010 +0000

    2010-03-29  Mikhail Naganov  <[e-mail]>
    
            Reviewed by Pavel Feldman.
    
            Remove a possibility of confusion from Profiles panel Welcome screen
            by turning buttons into non-clickable glyphs. Also, span instructions
            alongside panel width.
    
            https://bugs.webkit.org/show_bug.cgi?id=34319
    
            * English.lproj/localizedStrings.js:
            * inspector/front-end/ProfileView.js:
            (WebInspector.CPUProfileType.prototype.get welcomeMessage):
            * inspector/front-end/ProfilesPanel.js:
            (WebInspector.ProfilesPanel.prototype._addWelcomeMessage):
            * inspector/front-end/inspector.css:
            (.panel-enabler-view.welcome .instructions):
            (.panel-enabler-view.welcome button.status-bar-item):
            * src/js/HeapProfilerPanel.js:
            (WebInspector.HeapSnapshotProfileType.prototype.get welcomeMessage):