1
0
Fork 0
forked from wry/wry

seat: rename get_output to get_cursor_output

This commit is contained in:
khyperia 2025-12-23 14:57:02 +01:00 committed by Julian Orth
parent 2c03dbbaa4
commit a975e3b25a
10 changed files with 23 additions and 22 deletions

View file

@ -551,9 +551,9 @@ impl ConfigClient {
connector connector
} }
pub fn get_seat_workspace(&self, seat: Seat) -> Workspace { pub fn get_seat_cursor_workspace(&self, seat: Seat) -> Workspace {
let res = self.send_with_response(&ClientMessage::GetSeatWorkspace { seat }); let res = self.send_with_response(&ClientMessage::GetSeatCursorWorkspace { seat });
get_response!(res, Workspace(0), GetSeatWorkspace { workspace }); get_response!(res, Workspace(0), GetSeatCursorWorkspace { workspace });
workspace workspace
} }

View file

@ -372,7 +372,7 @@ pub enum ClientMessage<'a> {
MakeRenderDevice { MakeRenderDevice {
device: DrmDevice, device: DrmDevice,
}, },
GetSeatWorkspace { GetSeatCursorWorkspace {
seat: Seat, seat: Seat,
}, },
SetDefaultWorkspaceCapture { SetDefaultWorkspaceCapture {
@ -925,7 +925,7 @@ pub enum Response {
width: i32, width: i32,
height: i32, height: i32,
}, },
GetSeatWorkspace { GetSeatCursorWorkspace {
workspace: Workspace, workspace: Workspace,
}, },
GetDefaultWorkspaceCapture { GetDefaultWorkspaceCapture {

View file

@ -420,7 +420,7 @@ impl Seat {
/// ///
/// If no such workspace exists, `exists` returns `false` for the returned workspace. /// If no such workspace exists, `exists` returns `false` for the returned workspace.
pub fn get_workspace(self) -> Workspace { pub fn get_workspace(self) -> Workspace {
get!(Workspace(0)).get_seat_workspace(self) get!(Workspace(0)).get_seat_cursor_workspace(self)
} }
/// Returns the workspace that is currently active on the output that contains the seat's /// Returns the workspace that is currently active on the output that contains the seat's

View file

@ -1012,16 +1012,16 @@ impl ConfigProxyHandler {
self.state.double_click_distance.set(dist); self.state.double_click_distance.set(dist);
} }
fn handle_get_seat_workspace(&self, seat: Seat) -> Result<(), CphError> { fn handle_get_seat_cursor_workspace(&self, seat: Seat) -> Result<(), CphError> {
let seat = self.get_seat(seat)?; let seat = self.get_seat(seat)?;
let output = seat.get_output(); let output = seat.get_cursor_output();
let mut workspace = Workspace(0); let mut workspace = Workspace(0);
if !output.is_dummy if !output.is_dummy
&& let Some(ws) = output.workspace.get() && let Some(ws) = output.workspace.get()
{ {
workspace = self.get_workspace_by_name(&ws.name); workspace = self.get_workspace_by_name(&ws.name);
} }
self.respond(Response::GetSeatWorkspace { workspace }); self.respond(Response::GetSeatCursorWorkspace { workspace });
Ok(()) Ok(())
} }
@ -1056,7 +1056,7 @@ impl ConfigProxyHandler {
let name = self.get_workspace(ws)?; let name = self.get_workspace(ws)?;
let workspace = match self.state.workspaces.get(name.deref()) { let workspace = match self.state.workspaces.get(name.deref()) {
Some(ws) => ws, Some(ws) => ws,
_ => seat.get_output().create_workspace(name.deref()), _ => seat.get_cursor_output().create_workspace(name.deref()),
}; };
seat.set_workspace(&workspace); seat.set_workspace(&workspace);
Ok(()) Ok(())
@ -1112,7 +1112,8 @@ impl ConfigProxyHandler {
Some(ws) => ws, Some(ws) => ws,
_ => return Ok(()), _ => return Ok(()),
}, },
WorkspaceSource::Seat(s) => match self.get_seat(s)?.get_output().workspace.get() { WorkspaceSource::Seat(s) => match self.get_seat(s)?.get_cursor_output().workspace.get()
{
Some(ws) => ws, Some(ws) => ws,
_ => return Ok(()), _ => return Ok(()),
}, },
@ -2935,9 +2936,9 @@ impl ConfigProxyHandler {
ClientMessage::MakeRenderDevice { device } => self ClientMessage::MakeRenderDevice { device } => self
.handle_make_render_device(device) .handle_make_render_device(device)
.wrn("make_render_device")?, .wrn("make_render_device")?,
ClientMessage::GetSeatWorkspace { seat } => self ClientMessage::GetSeatCursorWorkspace { seat } => self
.handle_get_seat_workspace(seat) .handle_get_seat_cursor_workspace(seat)
.wrn("get_seat_workspace")?, .wrn("get_seat_cursor_workspace")?,
ClientMessage::GetSeatKeyboardWorkspace { seat } => self ClientMessage::GetSeatKeyboardWorkspace { seat } => self
.handle_get_seat_keyboard_workspace(seat) .handle_get_seat_keyboard_workspace(seat)
.wrn("get_seat_keyboard_workspace")?, .wrn("get_seat_keyboard_workspace")?,

View file

@ -457,7 +457,7 @@ impl WlSeatGlobal {
self.data_control_devices.remove(&device.id()); self.data_control_devices.remove(&device.id());
} }
pub fn get_output(&self) -> Rc<OutputNode> { pub fn get_cursor_output(&self) -> Rc<OutputNode> {
self.cursor_user_group.latest_output() self.cursor_user_group.latest_output()
} }

View file

@ -217,7 +217,7 @@ impl NodeSeatState {
.set_kb_node(&seat, seat.state.root.clone(), seat.state.next_serial(None)); .set_kb_node(&seat, seat.state.root.clone(), seat.state.next_serial(None));
// log::info!("keyboard_node = root"); // log::info!("keyboard_node = root");
if focus_last { if focus_last {
seat.get_output() seat.get_cursor_output()
.node_do_focus(&seat, Direction::Unspecified); .node_do_focus(&seat, Direction::Unspecified);
} }
} }

View file

@ -486,7 +486,7 @@ impl XdgToplevel {
if should_be_mapped { if should_be_mapped {
if !self.is_mapped.replace(true) { if !self.is_mapped.replace(true) {
if let Some(seat) = drag.source.data.seat.get() { if let Some(seat) = drag.source.data.seat.get() {
self.xdg.set_output(&seat.get_output()); self.xdg.set_output(&seat.get_cursor_output());
} }
self.toplevel_data.broadcast(self.clone()); self.toplevel_data.broadcast(self.clone());
self.tl_set_visible(self.state.root_visible()); self.tl_set_visible(self.state.root_visible());

View file

@ -136,7 +136,7 @@ impl XdgToplevelDragV1 {
if self.source.data.was_used() if self.source.data.was_used()
&& let Some(tl) = self.toplevel.get() && let Some(tl) = self.toplevel.get()
{ {
let output = seat.get_output(); let output = seat.get_cursor_output();
let (x, y) = seat.pointer_cursor().position(); let (x, y) = seat.pointer_cursor().position();
tl.drag.take(); tl.drag.take();
tl.after_toplevel_drag( tl.after_toplevel_drag(

View file

@ -60,7 +60,7 @@ impl ZwlrLayerShellV1RequestHandler for ZwlrLayerShellV1 {
self.client.lookup(req.output)?.global.clone() self.client.lookup(req.output)?.global.clone()
} else { } else {
for seat in self.client.state.seat_queue.rev_iter() { for seat in self.client.state.seat_queue.rev_iter() {
let output = seat.get_output(); let output = seat.get_cursor_output();
if !output.is_dummy { if !output.is_dummy {
break 'get_output output.global.opt.clone(); break 'get_output output.global.opt.clone();
} }

View file

@ -780,7 +780,7 @@ impl State {
pub fn ensure_map_workspace(&self, seat: Option<&Rc<WlSeatGlobal>>) -> Rc<WorkspaceNode> { pub fn ensure_map_workspace(&self, seat: Option<&Rc<WlSeatGlobal>>) -> Rc<WorkspaceNode> {
seat.cloned() seat.cloned()
.or_else(|| self.seat_queue.last().map(|s| s.deref().clone())) .or_else(|| self.seat_queue.last().map(|s| s.deref().clone()))
.map(|s| s.get_output()) .map(|s| s.get_cursor_output())
.or_else(|| self.root.outputs.lock().values().next().cloned()) .or_else(|| self.root.outputs.lock().values().next().cloned())
.or_else(|| self.dummy_output.get()) .or_else(|| self.dummy_output.get())
.unwrap() .unwrap()
@ -916,7 +916,7 @@ impl State {
let ws = match self.workspaces.get(name) { let ws = match self.workspaces.get(name) {
Some(ws) => ws, Some(ws) => ws,
_ => { _ => {
let output = output.unwrap_or_else(|| seat.get_output()); let output = output.unwrap_or_else(|| seat.get_cursor_output());
if output.is_dummy { if output.is_dummy {
log::warn!("Not showing workspace because seat is on dummy output"); log::warn!("Not showing workspace because seat is on dummy output");
return; return;
@ -929,7 +929,7 @@ impl State {
pub fn float_map_ws(&self) -> Rc<WorkspaceNode> { pub fn float_map_ws(&self) -> Rc<WorkspaceNode> {
if let Some(seat) = self.seat_queue.last() { if let Some(seat) = self.seat_queue.last() {
let output = seat.get_output(); let output = seat.get_cursor_output();
if !output.is_dummy { if !output.is_dummy {
return output.ensure_workspace(); return output.ensure_workspace();
} }