Skip to content

Commit 46e7382

Browse files
authored
Clarify log message (#430)
1 parent 91bd7f5 commit 46e7382

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

extensions/ql-vscode/src/query-history.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -223,11 +223,16 @@ export class QueryHistoryManager {
223223
if (queryHistoryItem.logFileLocation) {
224224
const uri = vscode.Uri.file(queryHistoryItem.logFileLocation);
225225
try {
226-
await vscode.window.showTextDocument(uri, {
227-
});
226+
await vscode.window.showTextDocument(uri);
228227
} catch (e) {
229228
if (e.message.includes('Files above 50MB cannot be synchronized with extensions')) {
230-
const res = await helpers.showBinaryChoiceDialog('File is too large to open in the editor, do you want to open it externally?');
229+
const res = await helpers.showBinaryChoiceDialog(
230+
`VS Code does not allow extensions to open files >50MB. This file
231+
exceeds that limit. Do you want to open it outside of VS Code?
232+
233+
You can also try manually opening it inside VS Code by selecting
234+
the file in the file explorer and dragging it into the workspace.`
235+
);
231236
if (res) {
232237
try {
233238
await vscode.commands.executeCommand('revealFileInOS', uri);

0 commit comments

Comments
 (0)