IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

[MM-56478] Only clear the suggestion pretext when the suggestion is completed and not when the list is cleared (#26151)

Browse Source 
* [MM-56478] Only clear the suggestion pretext when the suggestion is completed and not when the list is cleared

* Also clear pretext when unfocusing the box
This commit is contained in:
Devin Binnie 2024-02-12 11:05:11 -05:00 committed by GitHub
parent c723bf345a
commit cf11adff98
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
webapp/channels/src/components/suggestion/suggestion_box/suggestion_box.jsx
View File

@ -259,6 +259,7 @@ export default class SuggestionBox extends React.PureComponent {
handleEmitClearSuggestions = (delay = 0) => {
setTimeout(() => {
this.clear();
this.handlePretextChanged('');
}, delay);
};
@ -465,6 +466,7 @@ export default class SuggestionBox extends React.PureComponent {
}
this.clear();
this.handlePretextChanged('');
if (openCommandInModal) {
const appProvider = this.props.providers.find((p) => p.openAppsModalFromCommand);
@ -556,7 +558,6 @@ export default class SuggestionBox extends React.PureComponent {
selection: '',
suggestionBoxAlgn: undefined,
});
this.handlePretextChanged('');
}
};

8
webapp/channels/src/components/suggestion/suggestion_box/suggestion_box.test.jsx
View File

@ -293,4 +293,12 @@ describe('components/SuggestionBox', () => {
instance.componentDidMount();
expect(instance.handlePretextChanged).toHaveBeenCalledTimes(1);
});
test('should not clear pretext when clearing the suggestion list', () => {
const wrapper = shallow(<SuggestionBox {...baseProps}/>);
const instance = wrapper.instance();
instance.handlePretextChanged = jest.fn();
instance.clear();
expect(instance.handlePretextChanged).not.toHaveBeenCalled();
});
});