Skip to content
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

feat: add scrollBehavior option to Channel #676

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

AhyoungRyu
Copy link
Contributor

Addresses one of request in https://sendbird.atlassian.net/browse/AC-12

Added scrollBehavior prop to allow user to pass "smooth" option on scroll event via Channel component(provider).
But I set the default option to auto which is an initial value of this CSS property not to change the existing behavior.

You can see the difference of the each values of scrollBahavior in the mozilla docs site :)

@semgrep-sendbird
Copy link

Semgrep found 1 ssc-37ae9e0a-cbf0-4910-8f73-04f2275899a6 finding:

Risk: webpack 5.x before 5.76.0 is vulnerable to Improper Access Control due to ImportParserPlugin.js mishandling the magic comment feature. Due to this, webpack does not avoid cross-realm object access and an attacker who controls a property of an untrusted object can obtain access to the real global object.

Fix: Upgrade this library to at least version 5.76.0 at sendbird-uikit-react/samples/typescript_sample/package-lock.json:16391.

Reference(s): GHSA-hc6q-2mpp-qw7j, CVE-2023-28154

Ignore this finding from ssc-37ae9e0a-cbf0-4910-8f73-04f2275899a6.

@AhyoungRyu AhyoungRyu merged commit a4337a4 into main Jul 6, 2023
1 check passed
@AhyoungRyu AhyoungRyu deleted the feat/AC-12/scroll-behavior branch July 6, 2023 05:43
AhyoungRyu added a commit that referenced this pull request Jul 7, 2023
�Added a missing type `scrollBehavior` added by #676 to d ts file.
AhyoungRyu added a commit that referenced this pull request Jul 7, 2023
## [v3.6.3] (July 6 2023)

### Feat:
* Add new scrollBehavior prop to Channel (#676)
The default option is set to "auto," preserving the existing scroll
behavior.
  Possible to set smooth for smooth scroll effect.

### Fixes:
* Move message list scroll when the last message is edited (#674)
Added optional parameters to moveScroll to scroll only when the last
message reaches the bottom.
Scroll is now moved only when the updatedAt property of the last message
is changed.
* Add missing `UIKitConfig` to type definition (#677)
Reported by [GitHub PR
#650](#650 (comment)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants