rephrase the reset dialog for minecraft, say delete instead of clear for other targets

[srietkerk]

Fixes https://github.com/microsoft/pxt-minecraft/issues/2781

EDIT: made it so there's a target name in each reset dialog as suggested:

Arcade:

Microbit:

Minecraft:

[Copilot]

Pull Request Overview

Updates the reset dialog text to be more specific to the target platform, replacing "clear" with "delete" and providing Minecraft-specific messaging.

• Replaces generic "clear all projects" text with platform-specific messaging
• Uses "delete" instead of "clear" for better clarity
• Adds specific context for Minecraft users about MakeCode projects

Comments suppressed due to low confidence (1)

webapp/src/dialogs.tsx:720

• [nitpick] The variable name 'isMinecraft' could be more descriptive. Consider renaming to 'isMinecraftTarget' to be more explicit about what aspect is being checked.

    const isMinecraft = pxt.appTarget.id === "minecraft";

[thsparks]

If we end up needing to customize this for even more targets, we might consider moving it to pxtarget or targetconfig or something. But I'd say this is fine for now.

[Jaqster]

And this only pertains to local projects, right? So, I don't know if we want to do it in this dialog, but maybe we should point to some documentation about what Reset actually does (i.e. it doesn't delete projects in the cloud)

[abchatra]

I would suggest change this dialog to x at the top right, remove cancel and add a help button to point to documentation.
Personally, I don't like target checks in PXT. These error messages need localization, so moving to targetconfig/pxttarget is hard. So maybe just say "You are about to delete all MakeCode projects. Are you sure? This operation can't be undone." This would serve all targets.

[srietkerk]

I would suggest change this dialog to x at the top right, remove cancel and add a help button to point to documentation. Personally, I don't like target checks in PXT. These error messages need localization, so moving to targetconfig/pxttarget is hard. So maybe just say "You are about to delete all MakeCode projects. Are you sure? This operation can't be undone." This would serve all targets.

Sure, I can change the modal to add a link to documentation, but I do think cancel is important for a modal like this. Cancel should be a very obvious action so we don't worry users. But I don't think adding "MakeCode" to the dialog without added context is a safe change. I don't know how many of our Minecraft users also use our other targets, but if I saw something that said "you are about to delete all MakeCode projects", I would be worried that it means that my arcade and microbit projects would also be deleted.

[thsparks]

I would suggest change this dialog to x at the top right, remove cancel and add a help button to point to documentation. Personally, I don't like target checks in PXT. These error messages need localization, so moving to targetconfig/pxttarget is hard. So maybe just say "You are about to delete all MakeCode projects. Are you sure? This operation can't be undone." This would serve all targets.

Sure, I can change the modal to add a link to documentation, but I do think cancel is important for a modal like this. Cancel should be a very obvious action so we don't worry users. But I don't think adding "MakeCode" to the dialog without added context is a safe change. I don't know how many of our Minecraft users also use our other targets, but if I saw something that said "you are about to delete all MakeCode projects", I would be worried that it means that my arcade and microbit projects would also be deleted.

You could do something like "This will delete all MakeCode ${target} projects".

I agree I think cancel is good in this case. Nice to have a very clear "out" next to the big scary button.