Skip to content

Commit a770472

Browse files
authored
Merge pull request #41 from dev-opsss/fix-screenshot-notification
Replace dialog boxes with in-app notifications for missing screenshots
2 parents 0ada479 + eafbced commit a770472

File tree

1 file changed

+0
-34
lines changed

1 file changed

+0
-34
lines changed

electron/ProcessingHelper.ts

Lines changed: 0 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -248,15 +248,6 @@ export class ProcessingHelper {
248248
if (!screenshotQueue || screenshotQueue.length === 0) {
249249
console.log("No screenshots found in queue");
250250
mainWindow.webContents.send(this.deps.PROCESSING_EVENTS.NO_SCREENSHOTS);
251-
252-
// Show dialog if no screenshots
253-
dialog.showMessageBox(mainWindow, {
254-
type: 'info',
255-
title: 'No Screenshots Detected',
256-
message: 'No screenshots were found to process.',
257-
detail: 'Please take a screenshot first using Ctrl+H (or Cmd+H on Mac). Make sure your screenshot contains the coding problem you want to solve.',
258-
buttons: ['OK']
259-
});
260251
return;
261252
}
262253

@@ -265,15 +256,6 @@ export class ProcessingHelper {
265256
if (existingScreenshots.length === 0) {
266257
console.log("Screenshot files don't exist on disk");
267258
mainWindow.webContents.send(this.deps.PROCESSING_EVENTS.NO_SCREENSHOTS);
268-
269-
// Show error dialog
270-
dialog.showMessageBox(mainWindow, {
271-
type: 'warning',
272-
title: 'Screenshot Files Missing',
273-
message: 'The screenshot files were not found on disk.',
274-
detail: 'Try taking a new screenshot with Ctrl+H (or Cmd+H on Mac).',
275-
buttons: ['OK']
276-
});
277259
return;
278260
}
279261

@@ -365,14 +347,6 @@ export class ProcessingHelper {
365347
console.log("No extra screenshots found in queue");
366348
mainWindow.webContents.send(this.deps.PROCESSING_EVENTS.NO_SCREENSHOTS);
367349

368-
// Show dialog if no screenshots
369-
dialog.showMessageBox(mainWindow, {
370-
type: 'info',
371-
title: 'No Debug Screenshots',
372-
message: 'No screenshots were found for debugging.',
373-
detail: 'Please take screenshots of your code/errors with Ctrl+H before debugging.',
374-
buttons: ['OK']
375-
});
376350
return;
377351
}
378352

@@ -381,14 +355,6 @@ export class ProcessingHelper {
381355
if (existingExtraScreenshots.length === 0) {
382356
console.log("Extra screenshot files don't exist on disk");
383357
mainWindow.webContents.send(this.deps.PROCESSING_EVENTS.NO_SCREENSHOTS);
384-
385-
dialog.showMessageBox(mainWindow, {
386-
type: 'warning',
387-
title: 'Screenshot Files Missing',
388-
message: 'The debug screenshot files were not found.',
389-
detail: 'Try taking a new screenshot with Ctrl+H (or Cmd+H on Mac).',
390-
buttons: ['OK']
391-
});
392358
return;
393359
}
394360

0 commit comments

Comments
 (0)