Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WD-17111 - Implement new data/streaming page and form #1470

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pedoch
Copy link
Contributor

@pedoch pedoch commented Dec 18, 2024

Done

Implement new page: /data/streaming

QA

Issue / Card

@webteam-app
Copy link

@mattea-turic
Copy link
Collaborator

Thanks @pedoch ! A couple of comments:

For the hero section:

  • I noticed that the image is a little too light at the top (blends with bg), so I've added an updated asset here

For the "Streaming use cases" section:

  • Could you wrap the 50/50 subsection (text + video) in a shallow pls?
  • And use a muted hr above the grid of images

For the "Canonical solutions for data streaming" section:

  • On mobile, there's spacing lacking between the two pricing cards

For the "resources" section:

  • The hrs that don't span full-width should be muted

For the footer:

  • I think you've mistakenly duplicated the trademark text

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants