-
Notifications
You must be signed in to change notification settings - Fork 841
[Docs] AI overview guidelines #4113
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 15 commits
8362e4c
ffc5821
b0f90a6
9e6a7dc
1529b83
8045f54
22f7563
ac08377
7b476df
02aba14
a079ed4
7fde7ac
e7dff99
155b351
432b45b
9b7901a
b08eaf9
9d8cc8b
565017f
9a23d32
89b883f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
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.
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. This image shows the styling difference between core carbon and the AI presence. I don't think it necessarily needs to have different content between the two like we'd want with showing the "revert option." The other components in the image don't have real data either so if we were to do real data we would change it across all the components in this image. We'd lose the component name in the label if we did but that's the trade of of doing spec vs real content. Are we trying to help people to understand the structure of the component or how to use the component here? 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. Oh, I see your point. If I understand correctly, since we're showing the structure of the component, it's fine to go ahead with the Lorem Ipsum text. Thank you! |
aagonzales marked this conversation as resolved.
Show resolved
Hide resolved
|
aagonzales marked this conversation as resolved.
Show resolved
Hide resolved
|
aagonzales marked this conversation as resolved.
Show resolved
Hide resolved
|
aagonzales marked this conversation as resolved.
Show resolved
Hide resolved
|
aagonzales marked this conversation as resolved.
Show resolved
Hide resolved
|
Uh oh!
There was an error while loading. Please reload this page.