Web Inspector: add links to reference pages

• Component: Web Inspector
• Status: RESOLVED FIXED

[Devin Rousso]

Now that <https://webkit.org/web-inspector/> is live, we should create links to it in various places around Web Inspector:
 - Inspector Bootstrap Script default content
 - Local Resource popover
 - JavaScript/Event/URL breakpoint popovers
 - Device Settings popover
 - Settings Tab

[Devin Rousso]

Created attachment 387833 [details]
Patch

[Devin Rousso]

Created attachment 387835 [details]
Patch

[Devin Rousso]

Created attachment 387836 [details]
[Image] After Patch is applied

[Joseph Pecoraro]

The "Web Inspector Reference" link in the popover feels wrong. Was a (?) like icon considered for cases like this where the link and text take up a lot of space?

[Devin Rousso]

(In reply to Joseph Pecoraro from comment #4)
> The "Web Inspector Reference" link in the popover feels wrong. Was a (?) like icon considered for cases like this where the link and text take up a lot of space?
Tim suggested this internally.  I'm not opposed to it, but I do feel like it is less noticeable/understandable than text.  I'll give it a shot tonight and see what it looks like.

[Devin Rousso]

Created attachment 387979 [details]
Patch

[Devin Rousso]

Created attachment 387980 [details]
[Image] After Patch is applied

[Timothy Hatcher]

The (?) help icon looks better in the popover. I would also make the link text in settings be grey to match the goto arrow, so it fits in better.

[Devin Rousso]

Created attachment 387989 [details]
Patch

Adjusted the "Web Inspector Reference =>" link in the Settings Tab to be grey.

[WebKit Commit Bot]

Comment on attachment 387989 [details]
Patch

Clearing flags on attachment: 387989

Committed r254730: <https://trac.webkit.org/changeset/254730>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/58665440>