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

fix(Style guide): Modified the preview callout title #17826

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

nbaenam
Copy link
Contributor

@nbaenam nbaenam commented Jun 28, 2024

Updated the callout preview in the style guide and throughout the docs site.

@nbaenam nbaenam added content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers labels Jun 28, 2024
@nbaenam nbaenam requested a review from rhetoric101 June 28, 2024 12:50
@nbaenam nbaenam self-assigned this Jun 28, 2024
Copy link

Hi @nbaenam 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes).

@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues Jun 28, 2024
Copy link

netlify bot commented Jun 28, 2024

Deploy Preview for docs-website-netlify ready!

Name Link
🔨 Latest commit e48b318
🔍 Latest deploy log https://app.netlify.com/sites/docs-website-netlify/deploys/668485ccfe5112000804564a
😎 Deploy Preview https://deploy-preview-17826--docs-website-netlify.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@nbaenam nbaenam moved this from Hero to triage to Needs review in Docs PRs and Issues Jun 28, 2024
@nbaenam nbaenam removed the request for review from ally-sassman June 28, 2024 15:05
Copy link
Contributor

@ally-sassman ally-sassman left a comment

Choose a reason for hiding this comment

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

@nbaenam All of the callout titles should be uppercased, so <Callout title="preview"> should be <Callout title="Preview">

ally-sassman
ally-sassman previously approved these changes Jul 1, 2024
@rhetoric101
Copy link
Contributor

@nbaenam I'm working with the legal team to clarify our callout message. Let's not publish this until we get clarity on this from them.

@rhetoric101 rhetoric101 moved this from Needs review to Waiting on SME/Blocked in Docs PRs and Issues Jul 2, 2024
@rhetoric101 rhetoric101 moved this from Waiting on SME/Blocked to In progress in Docs PRs and Issues Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants