search

patternfly / virtual-assistant

This repo contains the code for the PatternFly virtual assistant extension
MIT License
0 stars 16 forks source link

Bot processing dots not animating #156

Open asuwebdesign opened 9 hours ago

asuwebdesign commented 9 hours ago

After a user sends a message and before the bot responds, 3 dots indicate the bot is processing the message. These dots are intended to animate with CSS transitions but are not showing.

rebeccaalpert commented 3 hours ago

We're not really allowing submission in any of the demos right now, but we should at least add documentation for this as part of the Message component. The Message component appears to be set up to show this if the isLoading prop is passed in. We don't appear to have docs on the Message component.

Nicole has a large PR up right now with a docs overhaul: https://github.com/patternfly/virtual-assistant/pull/151. We can address this after that merges.

It will be easier to troubleshoot any problems if there is an example in the docs.

asuwebdesign commented 3 hours ago

When I looked at the code, I think what I was actually seeing what the v5 virtual assistant's little loading thing instead of ours. So there might still be some lingering code we either refactor or deprecate. But I did see we have the animation styles in there. It's just something that came up in the hackathon.

Side note: we may split this visual cue into multiple variants later on, as several lightspeed teams are looking to stream their bot responses vs waiting for the entire response. So we might have an animated thing that trails at the end of the response like ChatGPT does today.

We can wait for the PR and revisit. Just documenting the observation today.