Bug 151373 - Web Inspector: Probes sidebar panel sections should use NavigationBar
Summary: Web Inspector: Probes sidebar panel sections should use NavigationBar
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Matt Baker
URL:
Keywords: InRadar
Depends on:
Blocks: 151377
  Show dependency treegraph
 
Reported: 2015-11-17 17:11 PST by Matt Baker
Modified: 2015-11-18 15:48 PST (History)
8 users (show)

See Also:


Attachments
[Patch] Proposed Fix (11.08 KB, patch)
2015-11-17 17:30 PST, Matt Baker
no flags Details | Formatted Diff | Diff
[Patch] Proposed Fix (13.14 KB, patch)
2015-11-17 19:16 PST, Matt Baker
no flags Details | Formatted Diff | Diff
[Patch] Proposed Fix (13.19 KB, patch)
2015-11-18 14:50 PST, Matt Baker
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Baker 2015-11-17 17:11:29 PST
* SUMMARY
Probes sidebar panel sections should use NavigationBar. Currently the add probe, clear samples, and remove probe buttons are added to the DOM manually, and lack tooltips and enabled/disabled state. The clear samples button should only be enabled when samples exist.
Comment 1 Radar WebKit Bug Importer 2015-11-17 17:12:21 PST
<rdar://problem/23586395>
Comment 2 Matt Baker 2015-11-17 17:30:36 PST
Created attachment 265721 [details]
[Patch] Proposed Fix
Comment 3 Matt Baker 2015-11-17 19:16:29 PST
Created attachment 265727 [details]
[Patch] Proposed Fix
Comment 4 BJ Burg 2015-11-18 13:16:05 PST
Comment on attachment 265727 [details]
[Patch] Proposed Fix

r=me
Comment 5 WebKit Commit Bot 2015-11-18 14:03:55 PST
Comment on attachment 265727 [details]
[Patch] Proposed Fix

Rejecting attachment 265727 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 265727, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
fs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 192583 = 2cdccd4e25f76b98954bfe31ba68605844034e38
r192584 = 355e1772eddb165f3350675b090fc6b42d9696df
r192585 = d17e14a3625201fac8271edf221660edd2917f38
r192586 = cb5552d0076102582c50977bd8e82ce3e244a4d3
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.webkit.org/results/446845
Comment 6 Matt Baker 2015-11-18 14:50:49 PST
Created attachment 265783 [details]
[Patch] Proposed Fix
Comment 7 WebKit Commit Bot 2015-11-18 15:47:57 PST
Comment on attachment 265783 [details]
[Patch] Proposed Fix

Clearing flags on attachment: 265783

Committed r192594: <http://trac.webkit.org/changeset/192594>
Comment 8 WebKit Commit Bot 2015-11-18 15:48:00 PST
All reviewed patches have been landed.  Closing bug.