Skip to content

Commit 804c708

Browse files
committed
Make the telemetry acceptance dialog modal
1 parent 2a24fad commit 804c708

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

extensions/ql-vscode/src/telemetry.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -146,9 +146,14 @@ export class TelemetryListener extends ConfigListener {
146146
result = await Promise.race([
147147
showBinaryChoiceDialog(
148148
'Do we have your permission to collect anonymous usage statistics help us improve CodeQL for VSCode? See [TELEMETRY.md](https://github.com/github/vscode-codeql/blob/main/.github/TELEMETRY.md)',
149-
false
149+
// TODO: We make this dialog modal for now. If users complain, then we make it non-modal
150+
// If we do decide to keep this dialog as modal, then this implementation can change and
151+
// we no longer need to call Promise.race. Before committing this PR, we need to make
152+
// this decision.
153+
true
150154
),
151-
new Promise(resolve => setTimeout(resolve, 10000))
155+
// Uncomment if we make the dialog non-modal
156+
// new Promise(resolve => setTimeout(resolve, 10000))
152157
]) as boolean | undefined;
153158
}
154159
if (result !== undefined) {

0 commit comments

Comments
 (0)