LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-10 13:00:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Refresh text-<input> contents when pressing backspace or delete

Browse Source 
Make sure to refresh the contents of text-<input> when pressing
backspace or delete key.

The methods 'handle_insert()' and 'handle_delete()' already had the call
to 'm_browsing_context.active_document()->force_layout()' so let us also
add it to 'handle_delete_character_after()'.
This commit is contained in:
Kenneth Myhra 2022-02-18 23:14:52 +01:00 committed by Andreas Kling
parent e2d797dd60
commit 4877e7593c
Notes: sideshowbarker 2024-07-17 18:33:33 +09:00 
Author: https://github.com/kennethmyhra
Commit: https://github.com/SerenityOS/serenity/commit/4877e7593c
Pull-request: https://github.com/SerenityOS/serenity/pull/12647
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Userland/Libraries/LibWeb/Page/EditEventHandler.cpp
View File

@ -33,6 +33,11 @@ void EditEventHandler::handle_delete_character_after(const DOM::Position& cursor
builder.append(text.substring_view(cursor_position.offset() + code_point_length));
node.set_data(builder.to_string());
// FIXME: When nodes are removed from the DOM, the associated layout nodes become stale and still
// remain in the layout tree. This has to be fixed, this just causes everything to be recomputed
// which really hurts performance.
m_browsing_context.active_document()->force_layout();
m_browsing_context.did_edit({});
}