Open richb-hanover opened 1 year ago
Further incentive to implement the fourth item above (clearing query pane should switch to SQL output tab)...
I was trying out one of the new Showcase queries. In the Playground, I cleared the query pane and pasted in the Showcase query. I was mystified that it caused an an error message. It "looked broken" to me until I remembered that I had to click the SQL Output tab.
Yes, that's https://github.com/PRQL/prql/issues/1779
As I try new queries in the Playground, I stumble on things... A few thoughts:
I wish there was better feedback for the "no pipeline" state than the red squiggly underline. It feels disconcerting that an empty page already has an "error". As a newcomer, I have the mental state, 'Uh oh. What am I supposed to type...?" And this state of anxiety remains if I type a comment (starting with
#
) or either of the two most common legal keywords (next point).Perhaps the left-hand pane should report the syntactic state of the PRQL query being built, while the right-hand pane indicates the "no pipeline" state. That way, a blank page (or several lines of comments) would indicate: "No SQL generated" on the right.
I wish there was better feedback when starting a valid query. Starting with the empty query and typing the following gives these errors:
let
- "Unknown name let"from
- "expected pipeline that resolves to a table..."Names of output tabs: Maybe the first tab could be "PRQL Output" (or "SQL Output") and the second could be "Chinook Output"? And the third and fourth be could be "PL Output" and "RQ Output"?
I like how Playground defaults to showing a real PRQL query and the results from the chinook database. And how typing additional PRQL commands affects the results. I also like how clearing out the PRQL query blanks the right-hand pane.
BUT... when I type a valid PRQL (say,
from foo
), the right-hand pane stays blank. I think the gesture of clearing out the PRQL pane should switch to the PRQL Output tab.Thanks!