Skip to content

Conversation

sravan-s
Copy link
Contributor

@sravan-s sravan-s commented Nov 2, 2022

Features:

  • For Channel component, added separate prop isLoading?.boolean Usage: <Channel channelUrl {currentChannelUrl} isLoading={!currentChannelUrl} />
    • For flicker in ChannelList, without extra props

Fixes:

  • React UIKit placeholder rendering issue
  • Fix scroll issue in ChannelList where user cannot load more channels
  • Modify TS interface getLeaveChannel to getLeaveGroupChannel in selectors

@sravan-s sravan-s requested a review from HoonBaek November 2, 2022 01:49
@sravan-s sravan-s self-assigned this Nov 2, 2022
@sravan-s sravan-s force-pushed the release/3.2.4 branch 2 times, most recently from 8a2fad1 to 61b61a5 Compare November 2, 2022 01:58
@sravan-s sravan-s force-pushed the release/3.2.4 branch 2 times, most recently from 9ebb394 to 7484fdd Compare November 2, 2022 02:43
Fix:
* React UIkit loading screen rendering issue
  - For Channel component, added seperate prop isLoading?.boolean
  Usage:
  ```
  <Channel channelUrl={currentChannelUrl} isLoading={!currentChannelUrl} />
  ```
  - For flicker in ChannelList, without extra props

* Fix scroll issue in ChannelList where user cannot load more channels
* Modify ts interface getLeaveChannel to getLeaveGroupChannel in selectors
@sravan-s sravan-s merged commit e48710f into stable/3.2.x Nov 2, 2022
@sravan-s sravan-s deleted the release/3.2.4 branch November 2, 2022 03:21
sravan-s pushed a commit that referenced this pull request Nov 2, 2022
Features:
* For Channel component, added separate prop isLoading?.boolean Usage: ```<Channel channelUrl {currentChannelUrl} isLoading={!currentChannelUrl} />```
  * For flicker in ChannelList, without extra props

Fixes:
* React UIKit placeholder rendering issue
* Fix scroll issue in ChannelList where user cannot load more channels
* Modify TS interface getLeaveChannel to getLeaveGroupChannel in selectors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants