Skip to content

feat(Chatbot): Move scrollbar for embedded - #868

Merged
rebeccaalpert merged 2 commits into
patternfly:mainfrom
rebeccaalpert:scroll
Aug 10, 2026
Merged

rebeccaalpert merged 2 commits into
patternfly:mainfrom
rebeccaalpert:scroll

Conversation

@rebeccaalpert

@rebeccaalpert rebeccaalpert commented Jul 10, 2026

Copy link
Copy Markdown
Member

Move scrollbar to far right for embedded layouts only if appropriate prop is used. See https://github.com/opendatahub-io/odh-dashboard/pull/8199/changes.

Fixes #886

@patternfly-build

patternfly-build commented Jul 10, 2026

Copy link
Copy Markdown

tabIndex={0}
aria-label={ariaLabel}
className={`pf-chatbot__messagebox ${position === 'bottom' ? 'pf-chatbot__messagebox--bottom' : ''} ${className ?? ''}`}
className={`pf-chatbot__messagebox ${position === 'bottom' ? 'pf-chatbot__messagebox--bottom' : ''} ${hasOuterScrollbar ? 'pf-chatbot__messagebox--outer-scroll' : ''} ${className ?? ''}`}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we import the css helper from Patternfly to create the className string?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All set!

Move scrollbar to far right for embedded layouts only.

Fixes patternfly#886
@rebeccaalpert
rebeccaalpert merged commit 6dbaaf2 into patternfly:main Aug 10, 2026
6 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 6.8.0-prerelease.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

aminabai added a commit to aminabai/odh-dashboard that referenced this pull request Aug 24, 2026
Bumping @patternfly/chatbot to 6.8.0-prerelease.8 pulled PatternFly 6.6
into a 6.5 Playground and Cypress could not find data-testid="chatbot".
Keep 6.5 and use the same layout CSS as patternfly/chatbot#868 locally.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MessageBox - Move scrollbar to far right on embedded

3 participants