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

Incorrect reference to Figma property #4152

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

Conversation

chadcwimberly
Copy link

The caption referenced "Auto Width" when the example was showing "Auto Height". I switched the text to say "Auto Height". I also fixed a few grammatically incorrect phrases in similar captions

Closes #

{{short description}}

Changelog

New

  • {{new thing}}

Changed

  • {{change thing}}

Removed

  • {{removed thing}}

The caption referenced "Auto Width" when the example was showing "Auto Height". I switched the text to say "Auto Height". I also fixed a few grammatically incorrect phrases in similar captions
Copy link

vercel bot commented Jul 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
carbondesignsystem ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 2, 2024 2:44pm

Copy link
Contributor

github-actions bot commented Jul 2, 2024

DCO Assistant Lite bot: Thanks for your submission! We ask that you sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by adding a comment below using this text:


I have read the DCO document and I hereby sign the DCO.


You can retrigger this bot by commenting recheck in this Pull Request

@@ -400,9 +400,9 @@ it to the top or bottom margin of a box:

</DoDont>

<DoDont type="dont" caption="In Figma, always use “Auto width” in the Text panel to ensure the text box is fitting with the content. Spacer snaps to the text box.">
<DoDont type="dont" caption="In Figma, always use “Auto Height” in the Text panel to ensure the text box fits with the content. Spacer snaps to the text box.">
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<DoDont type="dont" caption="In Figma, always use “Auto Height” in the Text panel to ensure the text box fits with the content. Spacer snaps to the text box.">
<DoDont type="dont" caption="In Figma, always use “Auto height” in the Text panel to ensure the text box fits with the content. Spacer snaps to the text box.">

Nice catch @chadcwimberly. Just suggesting this be in sentence case instead of title case like it is in Figma.

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.

None yet

2 participants