From 58dcbb215b560cd33beeb7a9d76bbcbf65f6c58f Mon Sep 17 00:00:00 2001 From: Elias Nahum Date: Tue, 3 Mar 2020 19:44:23 -0300 Subject: [PATCH] Fix typing on input box after returning from an archived channel (#3992) --- app/actions/views/channel.js | 1 - app/components/post_textbox/index.js | 1 - app/components/post_textbox/post_textbox.test.js | 1 - app/components/post_textbox/post_textbox_base.js | 6 ++---- 4 files changed, 2 insertions(+), 7 deletions(-) diff --git a/app/actions/views/channel.js b/app/actions/views/channel.js index 0254062cb..a297e4bce 100644 --- a/app/actions/views/channel.js +++ b/app/actions/views/channel.js @@ -318,7 +318,6 @@ export function selectPenultimateChannel(teamId) { lastChannel.delete_at === 0 && (lastChannel.team_id === teamId || isDMVisible || isGMVisible) ) { - dispatch(setChannelLoading(true)); dispatch(handleSelectChannel(lastChannelId)); return; } diff --git a/app/components/post_textbox/index.js b/app/components/post_textbox/index.js index 750fbc7e7..b4fe0934f 100644 --- a/app/components/post_textbox/index.js +++ b/app/components/post_textbox/index.js @@ -68,7 +68,6 @@ function mapStateToProps(state, ownProps) { channelTeamId: currentChannel ? currentChannel.team_id : '', canUploadFiles: canUploadFilesOnMobile(state), channelDisplayName: state.views.channel.displayName || (currentChannel ? currentChannel.display_name : ''), - channelIsLoading: state.views.channel.loading, channelIsReadOnly: isCurrentChannelReadOnly(state) || false, channelIsArchived: ownProps.channelIsArchived || (currentChannel ? currentChannel.delete_at !== 0 : false), currentUserId, diff --git a/app/components/post_textbox/post_textbox.test.js b/app/components/post_textbox/post_textbox.test.js index ba0f878bd..930e2ab7e 100644 --- a/app/components/post_textbox/post_textbox.test.js +++ b/app/components/post_textbox/post_textbox.test.js @@ -46,7 +46,6 @@ describe('PostTextBox', () => { channelId: 'channel-id', channelDisplayName: 'Test Channel', channelTeamId: 'channel-team-id', - channelIsLoading: false, channelIsReadOnly: false, currentUserId: 'current-user-id', deactivatedChannel: false, diff --git a/app/components/post_textbox/post_textbox_base.js b/app/components/post_textbox/post_textbox_base.js index 4afefd337..7ef2c4d07 100644 --- a/app/components/post_textbox/post_textbox_base.js +++ b/app/components/post_textbox/post_textbox_base.js @@ -76,7 +76,6 @@ export default class PostTextBoxBase extends PureComponent { channelId: PropTypes.string.isRequired, channelDisplayName: PropTypes.string, channelTeamId: PropTypes.string.isRequired, - channelIsLoading: PropTypes.bool, channelIsReadOnly: PropTypes.bool.isRequired, currentUserId: PropTypes.string.isRequired, deactivatedChannel: PropTypes.bool.isRequired, @@ -898,7 +897,7 @@ export default class PostTextBoxBase extends PureComponent { renderTextBox = () => { const {intl} = this.context; - const {channelDisplayName, channelIsArchived, channelIsLoading, channelIsReadOnly, theme, isLandscape, files, rootId} = this.props; + const {channelDisplayName, channelIsArchived, channelIsReadOnly, theme, isLandscape, files, rootId} = this.props; const style = getStyleSheet(theme); if (channelIsArchived) { @@ -906,7 +905,6 @@ export default class PostTextBoxBase extends PureComponent { } const {value, extraInputPadding} = this.state; - const textValue = channelIsLoading ? '' : value; const placeholder = this.getPlaceHolder(); let maxHeight = 150; @@ -939,7 +937,7 @@ export default class PostTextBoxBase extends PureComponent { >