a7cb874d382ecd4ff092b26163e4b6b043bd9a05
[WebKit-https.git] / Source / WebInspectorUI / ChangeLog
1 2018-01-04  Joseph Pecoraro  <pecoraro@apple.com>
2
3         Web Inspector: ⌘G / ⇧⌘G text search does not working after closing find banner
4         https://bugs.webkit.org/show_bug.cgi?id=181280
5         <rdar://problem/36291175>
6
7         Reviewed by Matt Baker.
8
9         * UserInterface/Views/ContentBrowser.js:
10         (WI.ContentBrowser.prototype._findBannerDidHide):
11         When the find banner is hidden trigger a new ContentView method,
12         searchHidden instead of the destructive searchCleared. This allows
13         most content views (text editors, DOM tree, Network Headers view)
14         to keep their populated search results.
15
16         * UserInterface/Views/ContentView.js:
17         (WI.ContentView.prototype.searchHidden):
18         * UserInterface/Views/LogContentView.js:
19         (WI.LogContentView.prototype.searchHidden):
20         By default searchHidden does nothing. The Console's LogContentView
21         treats the find banner differently and clears its search results.
22
23         * UserInterface/Views/TextEditor.js:
24         (WI.TextEditor.prototype.searchCleared):
25         Modernize some code while working in this area.
26
27         * UserInterface/Views/Main.css:
28         (.bouncy-highlight):
29         Ensure black text on yellow background in the bouncy highlight. In the
30         DOM Tree it could have been white if the find banner was closed.
31
32 2018-01-03  Ting-Wei Lan  <lantw44@gmail.com>
33
34         Replace hard-coded paths in shebangs with #!/usr/bin/env
35         https://bugs.webkit.org/show_bug.cgi?id=181040
36
37         Reviewed by Alex Christensen.
38
39         * Scripts/combine-resources.pl:
40         * Scripts/copy-user-interface-resources-dryrun.rb:
41         * Scripts/copy-user-interface-resources.pl:
42         * Scripts/fix-worker-imports-for-optimized-builds.pl:
43         * Scripts/remove-console-asserts-dryrun.rb:
44         * Scripts/remove-console-asserts.pl:
45         * Scripts/update-LegacyInspectorBackendCommands.rb:
46         * Scripts/update-codemirror-resources.rb:
47         * WebInspectorUI.vcxproj/build-webinspectorui.pl:
48
49 2018-01-03  Joseph Pecoraro  <pecoraro@apple.com>
50
51         Web Inspector: "Log Value" context menu is sometimes unavailable
52         https://bugs.webkit.org/show_bug.cgi?id=181278
53         <rdar://problem/36281649>
54
55         Reviewed by Devin Rousso.
56
57         * UserInterface/Views/ObjectPreviewView.css:
58         (.object-preview > .title):
59         * UserInterface/Views/ObjectTreeView.css:
60         (.object-tree.expanded > .title):
61         Make the expanded object title information 16px tall to match ObjectTree
62         tree element row heights. This eliminates the floating console message
63         location from overlapping the first ObjectTree's TreeElement and causing
64         truncation and other behavior issues (like Context Menu identification).
65
66 2018-01-03  Joseph Pecoraro  <pecoraro@apple.com>
67
68         Web Inspector: RTL - DOM Tree Element selection doesn't work
69         https://bugs.webkit.org/show_bug.cgi?id=181275
70         <rdar://problem/36290450>
71
72         Reviewed by Devin Rousso.
73
74         * UserInterface/Views/TreeOutline.js:
75         (WI.TreeOutline.prototype.treeElementFromEvent):
76         Provide a better explanation for why we are making the `x` adjustment here,
77         to detect the inner most tree element along the horizontal. Fix the algorithm
78         for RTL, since the intent is to adjust to the trailing edge of the container
79         which is on the opposite side in RTL.
80
81 2018-01-03  Joseph Pecoraro  <pecoraro@apple.com>
82
83         Web Inspector: Find banner sometimes does not work (when already populated and shown for first time on resource)
84         https://bugs.webkit.org/show_bug.cgi?id=181255
85         <rdar://problem/36248855>
86
87         Reviewed by Matt Baker.
88
89         * UserInterface/Views/TextEditor.js:
90         (WI.TextEditor.prototype.set string):
91         Defer any early searches until the initial content of a TextEditor has been set.
92         Such searches can happen when the FindBanner already has content when a
93         ContentView is first opened and needs to load its content from the backend.
94         Further, even though the content may be loaded from the backend before the
95         search results, microtask hops might cause the content to get to the TextEditor
96         after the search results.
97
98 2018-01-03  Joseph Pecoraro  <pecoraro@apple.com>
99
100         REGRESSION: Web Inspector: Debugger tab doesn't restore selected resource on reload
101         https://bugs.webkit.org/show_bug.cgi?id=181253
102         <rdar://problem/36280564>
103
104         Reviewed by Matt Baker.
105
106         * UserInterface/Views/DebuggerSidebarPanel.js:
107         (WI.DebuggerSidebarPanel.prototype.restoreStateFromCookie):
108         Add braces to ensure the trailing else is actually trailing the outer
109         chain as it was intended to be.
110
111 2018-01-02  Joseph Pecoraro  <pecoraro@apple.com>
112
113         Web Inspector: Clicking source location link in Console unexpectedly jumps to Network tab
114         https://bugs.webkit.org/show_bug.cgi?id=181229
115         <rdar://problem/36075219>
116
117         Reviewed by Matt Baker.
118
119         * UserInterface/Base/Main.js:
120         Cleanup linkifyURLAsNode. Ignore Search tab in generic handlePossibleLinkClick
121         when not already in the Search tab.
122
123         * UserInterface/Views/CallFrameView.js:
124         (WI.CallFrameView):
125         Ignore Search and Network tab in CallFrame links.
126
127         * UserInterface/Views/TabBrowser.js:
128         (WI.TabBrowser.prototype.bestTabContentViewForRepresentedObject):
129         Improve style.
130
131 == Rolled over to ChangeLog-2018-01-01 ==