-
Notifications
You must be signed in to change notification settings - Fork 99
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
DOC-279: add blog link and description to page #1401
Conversation
✅ Deploy Preview for hardcore-allen-f5257d ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for hardcore-allen-f5257d ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Text looks good.
The padding around the first image isn't ideal, but I see you've got another PR for that.
I think the call-out box is appropriate, it's just a little unfortunate that it's labelled "Tip" under the heading "Tips". Might be nice to change that heading. Not a big deal though.
Yeah, maybe just a note would be slightly better? I didn't want to introduce new nomenclature but agree Tip not ideal... |
No strong feelings on the best option. Change the heading, change the callout box to a note, or get rid of the callout box. If you did change the heading, you could go for "Best practice" or something like "Getting the most out of Ask AI" or "Help us help you". |
No description provided.