From 7affc455f2273aceb58ece58de79ffae2ff1ac53 Mon Sep 17 00:00:00 2001 From: Branden Cash <203336+ammmze@users.noreply.github.com> Date: Tue, 10 Dec 2024 14:28:50 -0700 Subject: [PATCH] fix(macro-prompts): preserve outer quotes (#2076) Previously if you had a macro prompt such as the following, the modal title would display `Question`, not `"Question"`. However, in other projects (Klipperscreen and Fluidd), they will preserve those outer quotes and they will be included in the title. This change makes Mainsail consistent with the other implementations. --- src/components/dialogs/TheMacroPrompt.vue | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/components/dialogs/TheMacroPrompt.vue b/src/components/dialogs/TheMacroPrompt.vue index d89a94328..a6eefa6f6 100644 --- a/src/components/dialogs/TheMacroPrompt.vue +++ b/src/components/dialogs/TheMacroPrompt.vue @@ -61,8 +61,7 @@ export default class TheMacroPrompt extends Mixins(BaseMixin) { private checkpointEvent: ServerStateEvent | null = null private currentPrompt: ServerStateEventPrompt[] = [] // regex that extracts the type and message, omitting the wrapping double quotes of the message (if any) - private promptMessageExp = - /^\/\/ action:prompt_(?[^\s]+) *(?:(?['"])(?.*)\k|(?.*))\s*$/ + private promptMessageExp = /^\/\/ action:prompt_(?[^\s]+) *(?.*)$/ get events() { return this.$store.state.server.events @@ -95,7 +94,7 @@ export default class TheMacroPrompt extends Mixins(BaseMixin) { break } - const message = (match?.groups?.msg1 || match?.groups?.msg2 || '').trim() + const message = (match?.groups?.msg || '').trim() // prepend the event to prompt events found in this chunk promptEvents.unshift({