Skip to content

Switch LLM overview contact section from preview iframe to Quarto listing#1281

Closed
nrichers wants to merge 1 commit intomainfrom
fix-cookie-consent-banner-preview
Closed

Switch LLM overview contact section from preview iframe to Quarto listing#1281
nrichers wants to merge 1 commit intomainfrom
fix-cookie-consent-banner-preview

Conversation

@nrichers
Copy link
Copy Markdown
Collaborator

@nrichers nrichers commented Apr 13, 2026

Pull Request Description

What and why?

Summary of changes

  • Replaces the embedded page preview (::: {.preview source=...}) on the Large language model features page with a standard Quarto listing tile, consistent with other docs (for example deployment pages that link to the contact URL).
  • Adds front matter listing configuration (id: llm-whats-next) with a single grid item: contact link, hero image, Request a demo title, and an LLM-focused description.
  • Wires the third What's next column to :::{#llm-whats-next} and removes the obsolete HTML comment placeholder.

Before: The contact call-to-action used an iframe-style preview of validmind.com/contact. After: The same destination is presented as a listing card (same pattern as multi-tenant / VPV "request a demo" tiles), which avoids embedding the live site and matches site-wide listing usage.

How to test

Try the preview:

https://docs-staging.validmind.ai/pr_previews/fix-cookie-consent-banner-preview/about/overview-llm-features.html#whats-next

Capto_ 2026-04-13_01-25-07_PM

What needs special review?

  • Copy on the listing description (LLM-focused line) and whether grid-columns: 1 in a narrow column looks acceptable on small viewports.

Dependencies, breaking changes, and deployment notes

  • None.

Release notes

Documentation: the LLM features overview page now uses a contact Request a demo listing tile instead of an embedded page preview.

Checklist

  • What and why
  • Screenshots or videos (Frontend)
  • How to test
  • What needs special review
  • Dependencies, breaking changes, and deployment notes
  • Labels applied
  • PR linked to Shortcut
  • Unit tests added (Backend)
  • Tested locally
  • Documentation updated (if required)
  • Environment variable additions/changes documented (if required)

@github-actions
Copy link
Copy Markdown
Contributor

Pull requests must include at least one of the required labels: internal (no release notes required), highlight, enhancement, bug, deprecation, documentation. Except for internal, pull requests must also include a description in the release notes section.

@github-actions
Copy link
Copy Markdown
Contributor

PR Summary

This PR introduces an enhancement to the large language model (LLM) features page by adding a new grid-based listing that highlights a call-to-action for requesting a demo. The changes include:

  • The insertion of a new front matter section labeled 'listing' that configures a grid layout. This grid is defined with one column and no sorting, and it contains a single entry linking to the contact page.
  • The listing entry includes details such as a path to the contact page, an image, a title ('Request a demo'), a brief description, and an author tag.
  • The grid block is integrated within the existing page layout, replacing a previously commented-out section. This ensures the demo request is visually emphasized using the newly introduced grid styling and identifiers.

These modifications aim to improve user interaction by making it easier for visitors to request a demo of the LLM-powered platform.

Test Suggestions

  • Verify the new grid listing renders correctly on the about page across various browsers and screen sizes.
  • Click the 'Request a demo' link to ensure it navigates to the correct contact page.
  • Ensure that the grid layout and its styling are responsive on mobile, tablet, and desktop devices.
  • Check that the image and description load properly without causing layout issues.

@nrichers nrichers added the internal Not to be externalized in the release notes label Apr 13, 2026
@nrichers nrichers requested a review from validbeck April 13, 2026 20:30
@github-actions
Copy link
Copy Markdown
Contributor

Validate docs site

✓ INFO: A live preview of the docs site is available — Open the preview

Copy link
Copy Markdown
Collaborator

@validbeck validbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, I patched this in a PR with just buttons but I can't decide which version I like better. 🤔 Maybe a combination of both (Since the Legal Ease actually looks really... scuffed) if we're going with the tile...

Image

I also kind of feel like the columns here don't make sense — for example, "request a demo" is the CTA for the 2nd column, but the tile itself is in a third column. I think it would make more sense to have "A commitment to transparency" (with the Legal Ease) button on it's own line ABOVE the "Try it yourself", then have the "Try it yourself" + demo CTA in the two column layout.

grid-columns: 1
sort: false
contents:
- path: https://validmind.com/contact/
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- path: https://validmind.com/contact/
- path: https://validmind.com/request-demo/

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hot take, this should actually take you to the request a demo page if we're going that route...

@nrichers
Copy link
Copy Markdown
Collaborator Author

Replaced by #1284.

@nrichers nrichers closed this Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Not to be externalized in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants