Bug 34201 – Web Inspector: Switching between Profiles and Resources puts Profiles into "welcome screen" mode.

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

[Pavel Feldman]

1. Start / stop profiling
2. Click on a resource link to get navigated to resources / scripts
3. Click on Profiler toolbar button to get back

Expected: your profile selected
Actual: profiler welcome screen

[Mikhail Naganov]

Created attachment 47519 [details]
proposed patch

[Pavel Feldman]

Comment on attachment 47519 [details]
proposed patch

> +    width: 0;

Should we use div instead?

[Mikhail Naganov]

Created attachment 47523 [details]
now using div

[Mikhail Naganov]

Created attachment 47529 [details]
assign more specialized class name to the div

[Timothy Hatcher]

Comment on attachment 47529 [details]
assign more specialized class name to the div

Why remove the image?

[Pavel Feldman]

(In reply to comment #5)
> (From update of attachment 47529 [details])
> Why remove the image?

Image is still there on the enabler panel. It is just that going from enabler panel to the welcome screen with the same image moving to the left was confusing.

[Timothy Hatcher]

Ok.

[Timothy Hatcher]

Comment on attachment 47529 [details]
assign more specialized class name to the div


> +.panel-enabler-view.welcome div.welcome-instructions-aligner {
> +    display: inline-block;
> +    width: 0;
>  }

A better way to hide the image is display: none.

[Mikhail Naganov]

(In reply to comment #8)
> (From update of attachment 47529 [details])
> 
> > +.panel-enabler-view.welcome div.welcome-instructions-aligner {
> > +    display: inline-block;
> > +    width: 0;
> >  }
> 
> A better way to hide the image is display: none.

The image is already removed by not creating 'img' node (WelcomeView.js:22--24.) The purpose of this div is to align 'instructions' div. 'instructions' has 'auto' height (because they have text content), and I couldn't find another way to align instructions vertically properly.

On enabler panel an image has fixed height, and alignment is done by using auto margins (and absolute positioning with top and bottom set to 0), then the text on the right is vertically centered relative to the image. On welcome view I'm proposing to use a zero-width div with fixed height for that purpose.

Do you know any better way for solving this?

[Timothy Hatcher]

I see that might be the only option then.

[WebKit Commit Bot]

Comment on attachment 47529 [details]
assign more specialized class name to the div

Rejecting patch 47529 from commit-queue.

[e-mail] does not have committer permissions according to http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/webkitpy/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in WebKitTools/Scripts/webkitpy/committers.py by adding yourself to the file (no review needed).  Due to bug 30084 the commit-queue will require a restart after your change.  Please contact [e-mail] to request a commit-queue restart.  After restart the commit-queue will correctly respect your committer rights.

[WebKit Commit Bot]

Comment on attachment 47529 [details]
assign more specialized class name to the div

Clearing flags on attachment: 47529

Committed r53996: <http://trac.webkit.org/changeset/53996>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.