RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 00:27:43 +00:00
Code Issues Activity Actions

LibGUI: Implement merging of TextDocument's RemoveTextCommand

Browse source 
When deleting text by pressing backspace, the commands will be merged
into a single RemoveTextCommand.
This commit is contained in:
Andreas Kling 2021-05-08 16:53:08 +02:00
parent ff6bac4854
commit 81bc861085
2 changed files with 19 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

18
Userland/Libraries/LibGUI/TextDocument.cpp
View file

@ -802,6 +802,24 @@ RemoveTextCommand::RemoveTextCommand(TextDocument& document, const String& text,
{
}
bool RemoveTextCommand::merge_with(GUI::Command const& other)
{
if (!is<RemoveTextCommand>(other))
return false;
auto& typed_other = static_cast<RemoveTextCommand const&>(other);
if (m_range.start() == typed_other.m_range.end()) {
// Merge backspaces
StringBuilder builder(m_text.length() + typed_other.m_text.length());
builder.append(typed_other.m_text);
builder.append(m_text);
m_text = builder.to_string();
m_range.set_start(typed_other.m_range.start());
return true;
}
// FIXME: Merge forward-deletes
return false;
}
void RemoveTextCommand::redo()
{
m_document.remove(m_range);