Use the display we have already

src/core/Logger.ts

@@ -104,32 +104,6 @@ export class Logger {
             this.handler({ time: new Date(), message, severity, logger: this, cause }, this.name);
         }
     }
-
-    showTrace = (message: string, cause?: any): void => {
-        this.showLog(Severity.Trace, message, cause);
-    };
-
-    showDebug = (message: string, cause?: any): void => {
-        this.showLog(Severity.Debug, message, cause);
-    };
-
-    showInfo = (message: string, cause?: any): void => {
-        this.showLog(Severity.Info, message, cause);
-    };
-
-    showWarn = (message: string, cause?: any): void => {
-        this.showLog(Severity.Warning, message, cause);
-    };
-
-    showError = (message: string, cause?: any): void => {
-        this.showLog(Severity.Error, message, cause);
-    };
-
-    //for graphically showing errors and other messages, assume we always want this to show, otherwise we wouldn't call it.
-    //Instead of doing an alert, we should have some better styled area, like a dismissable bar at the top of the screen with the error.
-    showLog(severity: Severity, message: string, cause?: any): void {
-        alert('Message: ' + message + '\r\n\r\nCause: ' + cause.message);
-    }
 }
 
 export class LogManager {

src/quest_editor/controllers/QuestEditorToolBarController.ts

@@ -21,16 +21,29 @@ import { convert_quest_from_model, convert_quest_to_model } from "../stores/mode
 import { LogManager } from "../../core/Logger";
 import { basename } from "../../core/util";
 import { Version } from "../../core/data_formats/parsing/quest/Version";
+import { WritableProperty } from "../../core/observable/property/WritableProperty";
+import { Result, failure } from "../../core/Result";
+import { Severity } from "../../core/Severity";
 
 const logger = LogManager.get("quest_editor/controllers/QuestEditorToolBarController");
 
 export type AreaAndLabel = { readonly area: AreaModel; readonly label: string };
 
 export class QuestEditorToolBarController extends Controller {
+    private readonly _result_dialog_visible = property(false);
+    private readonly _result: WritableProperty<Result<unknown> | undefined> = property(undefined);
+    private readonly _result_problems_message = property("");
+    private readonly _result_error_message = property("");
+
     private _save_as_dialog_visible = property(false);
     private _filename = property("");
     private _version = property(Version.BB);
 
+    readonly result_dialog_visible: Property<boolean> = this._result_dialog_visible;
+    readonly result: Property<Result<unknown> | undefined> = this._result;
+    readonly result_problems_message: Property<string> = this._result_problems_message;
+    readonly result_error_message: Property<string> = this._result_error_message;
+
     readonly vm_feature_active: boolean;
     readonly areas: Property<readonly AreaAndLabel[]>;
     readonly current_area: Property<AreaAndLabel>;
@@ -98,7 +111,7 @@ export class QuestEditorToolBarController extends Controller {
             quest_editor_store.quest_runner.running,
         );
 
-        this.can_step = quest_editor_store.quest_runner.paused;
+        this.can_step = quest_editor_store.quest_runner.paused;3
 
         this.can_stop = quest_editor_store.quest_runner.running;
 
@@ -182,7 +195,10 @@ export class QuestEditorToolBarController extends Controller {
                 quest && convert_quest_to_model(this.area_store, quest),
             );
         } catch (e) {
-            logger.showError("Couldn't read file.", e);
+            logger.error("Couldn't read file.", e);
+            this.set_result(
+                failure([{ severity: Severity.Error, ui_message: e.message }]),
+            );
         }
     };
 
@@ -270,4 +286,16 @@ export class QuestEditorToolBarController extends Controller {
     stop = (): void => {
         this.quest_editor_store.quest_runner.stop();
     };
+
+    dismiss_result_dialog = (): void => {
+        this._result_dialog_visible.val = false;
+    };
+
+    private set_result(result: Result<unknown>): void {
+        this._result.val = result;
+
+        if (result.problems.length) {
+            this._result_dialog_visible.val = true;
+        }
+    }
 }

src/quest_editor/gui/QuestEditorToolBarView.ts

@@ -15,6 +15,7 @@ import { Dialog } from "../../core/gui/Dialog";
 import { TextInput } from "../../core/gui/TextInput";
 import "./QuestEditorToolBarView.css";
 import { Version } from "../../core/data_formats/parsing/quest/Version";
+import { ResultDialog } from "../../core/gui/ResultDialog";
 
 export class QuestEditorToolBarView extends View {
     private readonly toolbar: ToolBar;
@@ -100,6 +101,14 @@ export class QuestEditorToolBarView extends View {
             icon_left: Icon.Stop,
             tooltip: "Stop execution (Shift-F5)",
         });
+        const dialog = this.disposable(
+            new ResultDialog({
+                visible: ctrl.result_dialog_visible,
+                result: ctrl.result,
+                problems_message: ctrl.result_problems_message,
+                error_message: ctrl.result_error_message,
+            }),
+        );
 
         const children = [
             new_quest_button,
@@ -219,6 +228,8 @@ export class QuestEditorToolBarView extends View {
 
             stop_button.onclick.observe(ctrl.stop),
             stop_button.enabled.bind_to(ctrl.can_stop),
+
+            dialog.ondismiss.observe(ctrl.dismiss_result_dialog),
         );
 
         this.finalize_construction();