- this->activate(*surface_id);
- this->state.main = this->state.sub = -1;
- this->state.s = LayoutState::Single;
- // commit changes
- this->controller->commit_changes();
- this->display->flush();
- return nullptr;
+
+ bool flush = false;
+ if (l.state.main != -1) {
+ this->deactivate(l.state.main);
+ l.state.main = -1;
+ flush = true;
+ }
+
+ if (l.state.sub != -1) {
+ this->deactivate(l.state.sub);
+ l.state.sub = -1;
+ flush = true;
+ }
+
+ l.state.s = LayoutState::Single;
+
+ if (flush) {
+ this->controller->commit_changes();
+ this->display->flush();
+ }
+ }
+
+ if (state.main == *surface_id || state.sub == *surface_id) {
+ return "Surface already active";