Skip to content

feat(HistoryDrawer): Add compact mode#523

Merged
nicolethoen merged 2 commits intopatternfly:mainfrom
rebeccaalpert:drawer-compact
Apr 22, 2025
Merged

feat(HistoryDrawer): Add compact mode#523
nicolethoen merged 2 commits intopatternfly:mainfrom
rebeccaalpert:drawer-compact

Conversation

@rebeccaalpert
Copy link
Member

@rebeccaalpert rebeccaalpert commented Apr 16, 2025

Add isCompact prop and associated styles to provide drawer with less spacing.

I'm breaking the density ticket into a series of smaller tickets so they'll be easier to review.

https://chatbot-pr-chatbot-523.surge.sh/patternfly-ai/chatbot/ui#drawer-with-search-and-new-chat-button

@patternfly-build
Copy link

patternfly-build commented Apr 16, 2025

@rebeccaalpert rebeccaalpert linked an issue Apr 16, 2025 that may be closed by this pull request
Copy link
Contributor

@edonehoo edonehoo left a comment

Choose a reason for hiding this comment

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

🤘

Copy link

@kaylachumley kaylachumley left a comment

Choose a reason for hiding this comment

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

compact looks good! I noticed that the alignment of the search bar is slightly off on the default version of the display drawer but the compact looks great. can we adjust the width search bar to line up with the close action and new chat action? also understand if we would need a new issue to do that.

compact with proper alignment:
Screenshot 2025-04-21 at 11 08 00 AM

default display with off alignment:
Screenshot 2025-04-21 at 11 07 31 AM

Add isCompact prop and associated styles to provide drawer with less spacing.
@rebeccaalpert
Copy link
Member Author

Just made the compact spacing global @kaylachumley - should get it where you want it!

@nicolethoen nicolethoen merged commit 15f36db into patternfly:main Apr 22, 2025
5 checks passed
@github-actions
Copy link

🎉 This PR is included in version 6.3.0-prerelease.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Control information density

6 participants