google / closure-compiler

A JavaScript checker and optimizer.
https://developers.google.com/closure/compiler/
Apache License 2.0
7.39k stars 1.15k forks source link

The GWT debugger options are confusing #1374

Open supersteves opened 8 years ago

supersteves commented 8 years ago

There are many checkboxes and they don't seem to correlate obviously to

All this is rather confusing. To make it slightly easier when trying to create a test case in the debugger, I suggest updating the CompilationParam enum to have additional properties which describe how they map to at least some of the above options, and setting a tooltip in DebuggerGwtMain.

I might do this in a spare moment if you indicate it will be accepted as a PR.

dimvar commented 8 years ago

We use the debugger to debug individual passes. If you want to repro something using simple or advanced optimizations, why can't you use http://closure-compiler.appspot.com?

supersteves commented 8 years ago

I use the debugger to:

... also I only discovered that other page yesterday!

I also don't know how to map the ~50 customisations we make (to CompilerOptions.java) to this page, e.g. specifying language in/out? If these are command-line options, I'm only familiar with the Java API.

All I'm suggesting is adding a tooltip to explain what Java API (and probably command-line option) each checkbox correlates to. And doing it myself. But only if it won't get rejected.

MatrixFrog commented 8 years ago

The debugger UX could definitely use some love. I don't see a reason to reject it, but I kinda feel like I'm still digging myself out from a pile of email after the holidays (which is why I haven't responded on several existing PRs), so it might take some time to get merged.