drracket icon indicating copy to clipboard operation
drracket copied to clipboard

Command-line arguments field should report invalid read syntax

Open samdphillips opened this issue 4 years ago • 3 comments

Inserting any text that is not a vector of strings is silently removed on closing the window.

If the text isn't valid then the user should be notified.

samdphillips avatar Aug 18 '20 18:08 samdphillips

A strategy that https://docs.racket-lang.org/drracket-cmdline-args/ is using is to parse (in this case it would be read-syntax) at every keystroke, and whenever it fails, change the background color. This gives an immediate feedback to users.

sorawee avatar Aug 18 '20 19:08 sorawee

I’d suggest it should just be text and default to —help.

The reasoning is that if a user is testing they want it to be close to what is entered at the CLI.

spdegabrielle avatar Aug 19 '20 00:08 spdegabrielle

I have what I believe is a better suggestion:

Remove the command line arguments from DrRacket codebase

If users need to create a command line application the plugin https://docs.racket-lang.org/drracket-cmdline-args/ is available. I'd suggest a link to the plugin from https://docs.racket-lang.org/reference/Command-Line_Parsing.html

spdegabrielle avatar Aug 20 '20 18:08 spdegabrielle