Skip to content

Commit 65df807

Browse files
committed
DevTools: Audits2 use built-in dialog overlay
BUG=845406,847350,846607 Change-Id: Id3ed220dcaf00cfbed5ba7b02e27412eab09b219 Reviewed-on: https://chromium-review.googlesource.com/1076325 Reviewed-by: Dmitry Gozman <[email protected]> Reviewed-by: Paul Irish <[email protected]> Commit-Queue: Paul Irish <[email protected]> Cr-Original-Commit-Position: refs/heads/master@{#562528}(cherry picked from commit 3fce93a) Reviewed-on: https://chromium-review.googlesource.com/1081069 Reviewed-by: Patrick Hulce <[email protected]> Cr-Commit-Position: refs/branch-heads/3440@{#65} Cr-Branched-From: 010ddcf-refs/heads/master@{#561733}
1 parent 498cfcc commit 65df807

File tree

5 files changed

+13
-14
lines changed

5 files changed

+13
-14
lines changed

third_party/blink/renderer/devtools/front_end/audits2/Audits2Panel.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ Audits2.Audits2Panel = class extends UI.Panel {
2929
this._controller.addEventListener(Audits2.Events.RequestAuditCancel, this._cancelAudit.bind(this));
3030

3131
this._renderToolbar();
32-
this.contentElement.createChild('div', 'audits2-dialog-overlay');
3332
this._auditResultsElement = this.contentElement.createChild('div', 'audits2-results-container');
3433
this._renderStartView();
3534

@@ -100,6 +99,9 @@ Audits2.Audits2Panel = class extends UI.Panel {
10099
this._startView.show(this.contentElement);
101100
this._startView.setUnauditableExplanation(this._unauditableExplanation);
102101
this._startView.setStartButtonEnabled(!this._unauditableExplanation);
102+
if (!this._unauditableExplanation)
103+
this._startView.focusStartButton();
104+
103105
this._newButton.setEnabled(false);
104106
this._refreshToolbarUI();
105107
this.setDefaultFocusedChild(this._startView);

third_party/blink/renderer/devtools/front_end/audits2/Audits2StartView.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,10 @@ Audits2.StartView = class extends UI.Widget {
142142
this.contentElement.style.overflow = 'auto';
143143
}
144144

145+
focusStartButton() {
146+
this._startButton.focus();
147+
}
148+
145149
/**
146150
* @param {boolean} isEnabled
147151
*/

third_party/blink/renderer/devtools/front_end/audits2/Audits2StatusView.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ Audits2.StatusView = class {
2323
this._scheduledFastFactTimeout = null;
2424

2525
this._dialog = new UI.Dialog();
26+
this._dialog.setDimmed(true);
27+
this._dialog.setCloseOnEscape(false);
2628
this._dialog.setOutsideClickCallback(event => event.consume(true));
2729
this._render();
2830
}

third_party/blink/renderer/devtools/front_end/audits2/audits2Dialog.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,10 @@
2222
text-align: left;
2323
}
2424

25+
.audits2-view button {
26+
z-index: 10;
27+
}
28+
2529
.audits2-status {
2630
width: 100%;
2731
flex: auto;

third_party/blink/renderer/devtools/front_end/audits2/audits2StartView.css

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -10,19 +10,6 @@
1010
line-height: 18px;
1111
}
1212

13-
.audits2-dialog-overlay {
14-
position: absolute;
15-
width: 100%;
16-
height: 100%;
17-
display: none;
18-
background: rgba(0,0,0,.3);
19-
z-index: 100;
20-
}
21-
22-
.in-progress .audits2-dialog-overlay {
23-
display: block;
24-
}
25-
2613
.audits2-start-view header {
2714
padding: 0 16px;
2815
display: flex;

0 commit comments

Comments
 (0)