Skip to content

Fix GraphiQL IDE rendering (#137)#138

Open
leonardwellthy wants to merge 1 commit intographql-python:mainfrom
leonardwellthy:fix/graphiql-rendering-issues-137
Open

Fix GraphiQL IDE rendering (#137)#138
leonardwellthy wants to merge 1 commit intographql-python:mainfrom
leonardwellthy:fix/graphiql-rendering-issues-137

Conversation

@leonardwellthy
Copy link
Contributor

  • Fix Flask views using Jinja2's render_template_string which HTML-escaped JSON values, breaking the GraphiQL JS config. Both sync and async views now use to_template_string() with the framework-agnostic simple_renderer.
  • Fix operationName not passed to the template due to a variable naming mismatch (operationName vs operation_name) in the sync Flask view.
  • Restore the locationQuery JS function accidentally removed in 578453f, which caused a ReferenceError when editing queries in the GraphiQL IDE.
  • Escape < and > as \u003c and \u003e in tojson() to prevent queries containing </script> from breaking the page by prematurely closing the script tag.
  • Add CodeMirror 5 fold gutter CSS to fix broken fold markers in the GraphiQL editor.

- Fix Flask views using Jinja2's render_template_string
  which HTML-escaped JSON values, breaking the GraphiQL
  JS config. Both sync and async views now use
  to_template_string() with the framework-agnostic
  simple_renderer.
- Fix operationName not passed to the template due to
  a variable naming mismatch (operationName vs
  operation_name) in the sync Flask view.
- Restore the locationQuery JS function accidentally
  removed in 578453f, which caused a ReferenceError
  when editing queries in the GraphiQL IDE.
- Escape < and > as \u003c and \u003e in tojson() to
  prevent queries containing </script> from breaking
  the page by prematurely closing the script tag.
- Add CodeMirror 5 fold gutter CSS to fix broken fold
  markers in the GraphiQL editor.
@leonardwellthy leonardwellthy force-pushed the fix/graphiql-rendering-issues-137 branch from 3388fc1 to 5cb34ef Compare February 6, 2026 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant