-
Notifications
You must be signed in to change notification settings - Fork 16
Open
Description
The do/while around this prompt() effectively disables the Cancel button and locks the user into a loop if they just want to get out. It's a bad UX.

PluginGenerator/JSPluginGen/JSPluginGen.js
Line 129 in a762457
| do |
For anyone stuck in this, I recommend replacing:
do
conf.name = prompt(
"How would you like your plugin to be called?",
"");
while (!conf.name || !conf.name.trim());
with
conf.name = prompt(
"How would you like your plugin to be called?",
"");
if(conf.name === null) return;
In the bigger picture though, this needs more explanation and a bit of refactoring.
- Don't put the prompt() on top of the initial alert().
- Open with a clear explanation of how this tool is to be used and what it does.
- The Readme text is unclear about what needs to be copied and then what the app does. The prompt() gets a name and then substitutes that name into the. js and html files. It is ALSO a tool to generate and test plugins from that code.
- I believe this would be more intuitive with a button to open the prompt, get the name, and do the substitutions.

Metadata
Metadata
Assignees
Labels
No labels