-
Notifications
You must be signed in to change notification settings - Fork 249
style(conversation-ai): define WHCM, typography utils, CSS patterns #6223
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
Changes from 9 commits
b37def5
7d21004
8c0ca1f
8840787
c78206a
b2f990a
e68f70c
45c428c
2abac81
ffecda6
414b2ff
cf4cfb8
3317825
5c9456d
2b0100e
1dfa05e
8f25c45
bfd2158
8d8104a
bdba0d9
2ab63a5
b725e89
d18af2b
ec1d836
aaa7b58
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
aramos-adobe marked this conversation as resolved.
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -42,6 +42,7 @@ | |
| gap: token("spacing-300"); | ||
| padding: token("spacing-300"); | ||
| background: token("background-layer-2-color"); | ||
| border: 1px solid transparent; | ||
| border-radius: token("corner-radius-800"); | ||
| box-shadow: token("drop-shadow-emphasized-default-x") token("drop-shadow-emphasized-default-y") token("drop-shadow-emphasized-default-blur") token("drop-shadow-emphasized-default-color"); | ||
| overflow: hidden; | ||
|
|
@@ -79,7 +80,7 @@ | |
| .swc-PromptField-artifacts { | ||
| display: flex; | ||
| flex-wrap: wrap; | ||
| gap: token("spacing-100"); | ||
| gap: token("spacing-300"); | ||
| align-items: start; | ||
| } | ||
|
|
||
|
|
@@ -92,17 +93,13 @@ | |
| flex: 0 0 auto; | ||
| } | ||
|
|
||
| /* Figma (node 2:2035): single card artifact is 324px wide. */ | ||
| .swc-PromptField-artifacts--single > slot::slotted([type="card"]) { | ||
| flex: 0 1 var(--swc-prompt-field-artifact-single-inline-size, 324px); | ||
| inline-size: min(100%, 324px); | ||
| max-inline-size: var(--swc-prompt-field-artifact-single-max-inline-size, 324px); | ||
| flex-basis: auto; | ||
|
aramos-adobe marked this conversation as resolved.
|
||
| } | ||
|
|
||
| /* Multi-artifact card layout targets 3+ cards per row responsively. */ | ||
| .swc-PromptField-artifacts--multiple > slot::slotted([type="card"]) { | ||
| flex: 1 1 var(--swc-prompt-field-artifact-multiple-inline-size, 160px); | ||
| min-inline-size: var(--swc-prompt-field-artifact-min-inline-size, 160px); | ||
| flex-basis: auto; | ||
| } | ||
|
|
||
| /* Prompt-field media artifact uses 68×68 preview tile. */ | ||
|
|
@@ -191,7 +188,7 @@ | |
| padding: 0; | ||
| color: token("gray-800"); | ||
| background: transparent; | ||
| border: none; | ||
| border: 1px solid transparent; | ||
| border-radius: token("corner-radius-500"); | ||
| cursor: pointer; | ||
| } | ||
|
|
@@ -223,7 +220,7 @@ | |
| padding: 0; | ||
| color: token("gray-25"); | ||
| background: token("neutral-background-color-default"); | ||
| border: none; | ||
| border: 1px solid transparent; | ||
| border-radius: token("corner-radius-full"); | ||
| cursor: pointer; | ||
| transition: background token("animation-duration-100") ease; | ||
|
|
@@ -238,7 +235,7 @@ | |
| padding: 0; | ||
| color: token("gray-25"); | ||
| background: token("gray-900"); | ||
| border: none; | ||
| border: 1px solid transparent; | ||
| border-radius: token("corner-radius-full"); | ||
| cursor: pointer; | ||
| transition: background token("animation-duration-100") ease; | ||
|
|
@@ -304,3 +301,7 @@ | |
| .swc-PromptField-legal-disclaimer a { | ||
| color: inherit; | ||
| } | ||
|
|
||
| .swc-PromptField-artifacts[hidden] { | ||
| display: none !important; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. just curious, do we need display: none on the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @rubencarvalho Yes we do. @5t3ph called this one out. This container renders in the DOM at zero height and full width before any artifacts is uploaded without this CSS modification.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yup! the browser rule on |
||
| } | ||
This file was deleted.
Uh oh!
There was an error while loading. Please reload this page.