Skip to content

Conversation

@HoonBaek
Copy link
Contributor

@HoonBaek HoonBaek commented Mar 9, 2023

For Internal Contributors

UIKIT-2872

Description Of Changes

  • Refactor
    • Replace FrozenNotification, UnreadCount (Also notification bar), and LoadingPlaceHolder into the MessageList component from ChannelUI
  • Bugfix
    • Do not scrollIntoBottom if the message is not reached the bottom
      scrollIntoBottom is a function moving scroll when a new message is received
    • Do not handleScroll when a message is initially mounted
      handleScroll is a function moving scroll when the last message's layout is changed

Types Of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply_

  • Bugfix
  • New feature
  • Documentation (correction or otherwise)
  • Cosmetics (whitespace, appearance (ex) Prettier)
  • Build configuration
  • Improvement (refactor code)

@HoonBaek HoonBaek requested a review from sravan-s March 9, 2023 11:14
@HoonBaek HoonBaek self-assigned this Mar 9, 2023
@HoonBaek HoonBaek changed the title Fix/UI kit 2872/keep context menu on message list fix: Keep scroll if context menu is opened when receiving messages Mar 9, 2023
@@ -1,3 +1,5 @@
export const MESSAGE_SCROLL_BUFFER = 10;
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

hasSeparator: boolean,
}

export const getMessagePartsInfo = ({
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@HoonBaek HoonBaek merged commit 1a5cf36 into main Mar 10, 2023
@HoonBaek HoonBaek deleted the fix/UIKIT-2872/Keep-context-menu-on-message-list branch March 10, 2023 01:13
@HoonBaek HoonBaek restored the fix/UIKIT-2872/Keep-context-menu-on-message-list branch May 25, 2023 04:24
@sravan-s sravan-s deleted the fix/UIKIT-2872/Keep-context-menu-on-message-list branch June 8, 2023 07:09
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.

4 participants