PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Remove all text when cutting in the composer

Browse source 
The previous function did in fact remove the elements, but left the model thinking there was a zero-length string. This approach deletes the text from the model and the DOM, resulting in the placeholder being shown when cutting all the text.

Part of https://github.com/vector-im/riot-web/issues/11378
This commit is contained in:
Travis Ralston 2020-01-15 19:08:14 -07:00
parent cad9562f8d
commit be3ef2b50e
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/components/views/rooms/BasicMessageComposer.js
View file

@ -209,8 +209,9 @@ export default class BasicMessageEditor extends React.Component {
const selectedParts = range.parts.map(p => p.serialize()); const selectedParts = range.parts.map(p => p.serialize());
event.clipboardData.setData("application/x-riot-composer", JSON.stringify(selectedParts)); event.clipboardData.setData("application/x-riot-composer", JSON.stringify(selectedParts));
if (type === "cut") { if (type === "cut") {
selection.deleteFromDocument(); // Remove the text from the composer
range.replace([]); const {caret} = getCaretOffsetAndText(this._editorRef, selection);
this.props.model.update("", event.inputType, caret);
} }
event.preventDefault(); event.preventDefault();
} }