docs(AI): Updates AI resources with latest guidance.#4967
Open
edonehoo wants to merge 2 commits intopatternfly:mainfrom
Open
docs(AI): Updates AI resources with latest guidance.#4967edonehoo wants to merge 2 commits intopatternfly:mainfrom
edonehoo wants to merge 2 commits intopatternfly:mainfrom
Conversation
Collaborator
phcox
reviewed
Mar 24, 2026
phcox
left a comment
There was a problem hiding this comment.
Hey Erin,
I concentrated mostly on the new Design Language page, but I did read everything else and it all looked good to me.
Comments on the Design Language page:
- The images aren't being displayed very well. The resolution is off and they look grainy. The icons are somewhat better but even they seem a little off.
- It seems like the two robot icon variants should be presented when they are being described, maybe even in context to better differentiate them.
- Typo in this sentence: "Do not communicate the use of AI through icons alone. Ensure they are also paited with a text label or tooltip."
- This sentence may not be entirely accurate: "Because avatars have personality and don't have interactive state changes, you can adjust the background colors of the standard robot icon, including adding soft gradients or drop shadows to create depth." I think the colors will be limited to either PF non-status or RH brand color for non-interactivity, but that conversation just got brought up again in our design language meeting. Kayla is going to talk to Andrew on 3/25 to try to come to a resolution. Regardless, we don't want to say that soft gradients and drop shadows are OK - we backed off that as we iterated on the deck.
- This sentence: "There are different icon variations and animations available to show thinking states, react to content, or differentiate chatbot options." This isn't true at the moment. It's something we'll look at in the future. I suggest removing it.
- This sentence: "AI is represented by a sparkle icon, which is square with slightly rounded corners." This is probably just a personal preference, but I don't think of that sparkle as being square; to me it's more like a diamond with concave sides.
That's all I've got. Looks really good!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #4621 and #4792
Will require a follow-up in chatbot repo to add back some info I'm removing from these docs (handling unsafe user requests, message streaming)