-
Notifications
You must be signed in to change notification settings - Fork 5
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
Blog sidebar rework #717
Blog sidebar rework #717
Conversation
Visit the preview URL for this PR (updated for commit dad2e79): https://estuary-marketing--pr717-brenosalv-feature-65-u52r10hl.web.app (expires Sat, 15 Mar 2025 05:40:46 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 76f6b095a0752e5d9c6c890267f9fdc3e392161e |
…to Brenosalv/feature/654-blog-sidebar-rework
<OutboundLinkFilled href={dashboardRegisterUrl}> | ||
Start Building For Free | ||
</OutboundLinkFilled> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The other link like this opens in a new tab. I would think this would as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in PR #688
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
#654
Changes on the sidebar from the blog post and comparison pages:
Tests / Screenshots
TOC on the blog post page:

TOC on the comparison page:
