-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Fix 6428/chat open on clicking dropdown #6429
Fix 6428/chat open on clicking dropdown #6429
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
The pull request addresses the issue of the chat opening when clicking the dropdown by replacing the SupportChat
component with the SupportDropdown
component and making related updates.
- AppNavigationDrawer.tsx: Replaced
SupportChat
withSupportDropdown
in the navigation drawer footer. - SupportButton.tsx: Renamed from
SupportChat
, addedisClickAble
prop, and updated the onClick handler. - SupportButtonSkeletonLoader.tsx: Renamed from
SupportChatSkeletonLoader
for clarity. - SupportDropdown.tsx: Introduced a new
SupportDropdown
component with options for 'Talk to us' and 'Documentation'. - SettingsBilling.tsx: Replaced
SupportChat
withSupportButton
and set it todisplay: none
to prevent interference.
7 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings
dropdownId={dropdownId} | ||
dropdownPlacement="top-start" | ||
dropdownOffset={{ x: 0, y: -28 }} | ||
clickableComponent={<SupportButton isClickAble={false} />} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Style: Consider renaming isClickAble
to isClickable
for consistency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
Thanks @NitinPSingh for your contribution! |
This issue was caused due to last comomit on issue by me only fix_6127/support_button_updated
I had fixed the chat opening problem , in this commit