The Guided reports "Show SQL" e.g. https://intranet.bywatersolutions.com/cgi-bin/koha/reports/guided_reports.pl?reports=120&phase=Show%20SQL Looks very similar to the "Edit SQL" screen. https://intranet.bywatersolutions.com/cgi-bin/koha/reports/guided_reports.pl?reports=120&phase=Edit%20SQL Because of the visual similarity, there is a strong temptation to edit the SQL on the Show SQL screen. This is an annoyance; it would be better UX to display the SQL in a <code> tag.
I wonder if this is improved by the switch to CodeMirror for viewing and editing SQL?
(In reply to Owen Leonard from comment #1) > I wonder if this is improved by the switch to CodeMirror for viewing and > editing SQL? I think it's still valid, if you go to: http://localhost:8081/cgi-bin/koha/reports/guided_reports.pl?reports=1&phase=Show%20SQL It does look like the edit screen and you can put your cursor into it, even if you cannot type. Not sure if there is a 'just display' mode for codemirror?
I feel like the modal with the edit button below is probably making this clear enough now.