eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-05-03 04:52:06 +02:00
Code Issues Packages Projects Releases Wiki Activity

HackStudio: Resize editors when removing panes

Browse Source 
Splitter does weird things when you resize and then remove children.
This works around the limitation by forcing at least one of the editors
to fill the space. It's janky, but at least doesn't result in the last
editor not filling the window.
This commit is contained in:
Nicholas Hollett
2021-07-26 21:20:07 +01:00
committed by Andreas Kling
parent e76af0fe16
commit 4857943a71
Notes: sideshowbarker 2024-07-18 08:17:28 +09:00 
Author: https://github.com/niax
Commit: https://github.com/SerenityOS/serenity/commit/4857943a712
Pull-request: https://github.com/SerenityOS/serenity/pull/9030
Issue: https://github.com/SerenityOS/serenity/issues/8817
1 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
Userland/DevTools/HackStudio/HackStudioWidget.cpp
View File

@@ -553,6 +553,18 @@ NonnullRefPtr<GUI::Action> HackStudioWidget::create_remove_current_editor_action
auto wrapper = m_current_editor_wrapper;
m_switch_to_next_editor->activate();
m_editors_splitter->remove_child(*wrapper);
auto child_editors = m_editors_splitter->child_widgets();
bool has_child_to_fill_space = false;
for (auto& editor : child_editors) {
if (editor.max_height() == -1) {
has_child_to_fill_space = true;
break;
}
}
if (!has_child_to_fill_space)
child_editors.last().set_max_height(-1);
m_all_editor_wrappers.remove_first_matching([&wrapper](auto& entry) { return entry == wrapper.ptr(); });
update_actions();
});