Skip to content

Keep highlighted blocks on the grouping page from becoming query cells - #4935

Merged
sebastiannberg merged 2 commits into
masterfrom
johsol/tensor-typo-grouping-yql
Oct 8, 2026
Merged

sebastiannberg merged 2 commits into
masterfrom
johsol/tensor-typo-grouping-yql

Conversation

@johansolbakken

Copy link
Copy Markdown
Contributor

The grouping page turns each <pre> that contains | into an executable grouping cell, unless it is marked no-transform. A highlighted block renders its own <pre> inside the marked one, so the schema product example (indexing: attribute | summary) was shown as a broken query cell, starting at summary } } field sample_bucket ….

  • Skip <pre> elements inside a no-transform element or a highlighted block. The executable cells are plain <pre> blocks and are not affected.
  • Highlight the five query and continuation examples on the page that are not executable, as vespa-yql.
  • Fix tensor(id{}} → tensor(id{}) in the tensor examples.

Checked in a browser: the page now has 63 executable cells, all grouping queries (64 before, one of them the schema), and the schema example is shown highlighted.

🤖 Generated with Claude Code

johansolbakken and others added 2 commits October 8, 2026 16:04
The grouping page turns each <pre> with a "|" into an executable grouping
cell, unless it is marked no-transform. A highlighted block has its own
<pre> inside the marked one, so the schema example with "indexing: ... |"
was shown as a broken query cell. Skip <pre> elements inside a
no-transform element or a highlighted block.

Highlight the query and continuation examples that are not executable.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sebastiannberg
sebastiannberg merged commit 5e6c6b0 into master Oct 8, 2026
2 checks passed
@sebastiannberg
sebastiannberg deleted the johsol/tensor-typo-grouping-yql branch October 8, 2026 18:45
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.

2 participants