-
Notifications
You must be signed in to change notification settings - Fork 7
feat(chat): add chat component #1688
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
base: master
Are you sure you want to change the base?
Conversation
…onents into dmdimitrov/chat-ai-component
…onents into dmdimitrov/chat-ai-component
…onents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
* fix(*): Scroll typing indicator/suggestions into view. * refactor(*); Cleaned up scroll to bottom logic
* Segregate renderers into explicit containers to keep rendering hierarchy in React applications using Portal. * Tweaked chat types a bit. * Started to restructure unit test suite so it doesn't care that much for exact DOM representation.
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
…iteUI/igniteui-webcomponents into dmdimitrov/chat-ai-component
Issues Found in Chat Component@SisIvanova @igdmdimitrov Those are some of the observations I have from using the component inside the Stream-Manager app Auto-scrolling disrupts readingProblem: Expected behavior: Empty message-header/message-attachments/message-actions containers remain in the DOMProblem: Expected behavior:
No customization for message container spacingProblem: I had to override the component to remove the default padding and margin on the message container. Other developers may face the same issue if they have custom layouts. Solution: Provide CSS custom properties (variables) for message container padding and margin. This allows consumers to adjust spacing without resorting to overrides. |
No description provided.