mirror of
				https://github.com/swaywm/sway.git
				synced 2025-11-03 09:01:43 -05:00 
			
		
		
		
	tree/view: don't give focus to views mapped under fullscreen views
Fixes #6211.
(cherry picked from commit 80128d23ba)
			
			
This commit is contained in:
		
							parent
							
								
									fc1cce9348
								
							
						
					
					
						commit
						b51721901d
					
				
					 1 changed files with 11 additions and 8 deletions
				
			
		| 
						 | 
				
			
			@ -600,6 +600,11 @@ static bool should_focus(struct sway_view *view) {
 | 
			
		|||
		return true;
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	// View opened "under" fullscreen view should not be given focus.
 | 
			
		||||
	if (root->fullscreen_global || map_ws->fullscreen) {
 | 
			
		||||
		return false;
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	// Views can only take focus if they are mapped into the active workspace
 | 
			
		||||
	if (prev_ws != map_ws) {
 | 
			
		||||
		return false;
 | 
			
		||||
| 
						 | 
				
			
			@ -807,8 +812,8 @@ void view_map(struct sway_view *view, struct wlr_surface *wlr_surface,
 | 
			
		|||
	if (wlr_surface_is_xwayland_surface(wlr_surface)) {
 | 
			
		||||
		struct wlr_xwayland_surface *xsurface =
 | 
			
		||||
				wlr_xwayland_surface_from_wlr_surface(wlr_surface);
 | 
			
		||||
    	set_focus = (wlr_xwayland_icccm_input_model(xsurface) !=
 | 
			
		||||
			WLR_ICCCM_INPUT_MODEL_NONE) && set_focus;
 | 
			
		||||
		set_focus &= wlr_xwayland_icccm_input_model(xsurface) !=
 | 
			
		||||
				WLR_ICCCM_INPUT_MODEL_NONE;
 | 
			
		||||
	}
 | 
			
		||||
#endif
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -819,11 +824,9 @@ void view_map(struct sway_view *view, struct wlr_surface *wlr_surface,
 | 
			
		|||
	const char *app_id;
 | 
			
		||||
	const char *class;
 | 
			
		||||
	if ((app_id = view_get_app_id(view)) != NULL) {
 | 
			
		||||
		wlr_foreign_toplevel_handle_v1_set_app_id(
 | 
			
		||||
				view->foreign_toplevel, app_id);
 | 
			
		||||
		wlr_foreign_toplevel_handle_v1_set_app_id(view->foreign_toplevel, app_id);
 | 
			
		||||
	} else if ((class = view_get_class(view)) != NULL) {
 | 
			
		||||
		wlr_foreign_toplevel_handle_v1_set_app_id(
 | 
			
		||||
				view->foreign_toplevel, class);
 | 
			
		||||
		wlr_foreign_toplevel_handle_v1_set_app_id(view->foreign_toplevel, class);
 | 
			
		||||
	}
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue