Extend the fullscreen command
Patch status: needinfo
Patch by Mats
Long description:
Rather than just toggling the fullscreen modes, allow to set them directly with: fullscreen enable|toggle [global] fullscreen disable For compatibility, retain the previous command and its toggling behavior: fullscreen [global] fixes #1120
To apply this patch, use:
curl http://cr.i3wm.org/patch/663/raw.patch | git am
b/docs/userguide
33 |
@@ -91,7 +91,7 @@ To display a window in fullscreen mode or to go out of fullscreen mode again, |
34 |
press +$mod+f+. |
35 |
|
36 |
There is also a global fullscreen mode in i3 in which the client will span all |
37 |
-available outputs (the command is +fullscreen global+). |
38 |
+available outputs (the command is +fullscreen toggle global+). |
39 |
|
40 |
=== Opening other applications |
41 |
|
42 |
@@ -367,7 +367,7 @@ bindcode [--release] [Modifiers+]keycode command |
43 |
*Examples*: |
44 |
-------------------------------- |
45 |
# Fullscreen |
46 |
-bindsym $mod+f fullscreen |
47 |
+bindsym $mod+f fullscreen toggle |
48 |
|
49 |
# Restart |
50 |
bindsym $mod+Shift+r restart |
51 |
@@ -1502,9 +1502,13 @@ Use +layout toggle split+, +layout stacking+, +layout tabbed+, +layout splitv+ |
52 |
or +layout splith+ to change the current container layout to splith/splitv, |
53 |
stacking, tabbed layout, splitv or splith, respectively. |
54 |
|
55 |
-To make the current window (!) fullscreen, use +fullscreen+, to make |
56 |
-it floating (or tiling again) use +floating enable+ respectively +floating disable+ |
57 |
-(or +floating toggle+): |
58 |
+To make the current window (!) fullscreen, use +fullscreen enable+ (or |
59 |
++fullscreen enable global+ for the global mode), to leave either fullscreen |
60 |
+mode use +fullscreen disable+, and to toggle between these two states use |
61 |
++fullscreen toggle+ (or +fullscreen toggle global+). |
62 |
+ |
63 |
+Likewise, to make the current window floating (or tiling again) use +floating |
64 |
+enable+ respectively +floating disable+ (or +floating toggle+): |
65 |
|
66 |
*Syntax*: |
67 |
-------------- |
68 |
@@ -1525,7 +1529,7 @@ bindsym $mod+x layout toggle |
69 |
bindsym $mod+x layout toggle all |
70 |
|
71 |
# Toggle fullscreen |
72 |
-bindsym $mod+f fullscreen |
73 |
+bindsym $mod+f fullscreen toggle |
74 |
|
75 |
# Toggle floating/tiling |
76 |
bindsym $mod+t floating toggle |
b/i3.config
81 |
@@ -75,7 +75,7 @@ bindsym Mod1+h split h |
82 |
bindsym Mod1+v split v |
83 |
|
84 |
# enter fullscreen mode for the focused container |
85 |
-bindsym Mod1+f fullscreen |
86 |
+bindsym Mod1+f fullscreen toggle |
87 |
|
88 |
# change container layout (stacked, tabbed, toggle split) |
89 |
bindsym Mod1+s layout stacking |
b/i3.config.keycodes
94 |
@@ -69,7 +69,7 @@ bindcode $mod+43 split h |
95 |
bindcode $mod+55 split v |
96 |
|
97 |
# enter fullscreen mode for the focused container |
98 |
-bindcode $mod+41 fullscreen |
99 |
+bindcode $mod+41 fullscreen toggle |
100 |
|
101 |
# change container layout (stacked, tabbed, toggle split) |
102 |
bindcode $mod+39 layout stacking |
b/include/commands.h
107 |
@@ -187,10 +187,10 @@ void cmd_focus_level(I3_CMD, char *level); |
108 |
void cmd_focus(I3_CMD); |
109 |
|
110 |
/** |
111 |
- * Implementation of 'fullscreen [global]'. |
112 |
+ * Implementation of 'fullscreen [enable|disable|toggle] [global]'. |
113 |
* |
114 |
*/ |
115 |
-void cmd_fullscreen(I3_CMD, char *fullscreen_mode); |
116 |
+void cmd_fullscreen(I3_CMD, char *action, char *fullscreen_mode); |
117 |
|
118 |
/** |
119 |
* Implementation of 'move <direction> [<pixels> [px]]'. |
b/include/con.h
124 |
@@ -173,6 +173,18 @@ void con_fix_percent(Con *con); |
125 |
void con_toggle_fullscreen(Con *con, int fullscreen_mode); |
126 |
|
127 |
/** |
128 |
+ * Enables fullscreen mode for the given container, if necessary. |
129 |
+ * |
130 |
+ */ |
131 |
+void con_enable_fullscreen(Con *con, fullscreen_mode_t fullscreen_mode); |
132 |
+ |
133 |
+/** |
134 |
+ * Disables fullscreen mode for the given container, if necessary. |
135 |
+ * |
136 |
+ */ |
137 |
+void con_disable_fullscreen(Con *con); |
138 |
+ |
139 |
+/** |
140 |
* Moves the given container to the currently focused container on the given |
141 |
* workspace. |
142 |
* |
b/man/i3.man
147 |
@@ -230,7 +230,7 @@ bindsym Mod1+h split h |
148 |
bindsym Mod1+v split v |
149 |
|
150 |
# enter fullscreen mode for the focused container |
151 |
-bindsym Mod1+f fullscreen |
152 |
+bindsym Mod1+f fullscreen toggle |
153 |
|
154 |
# change container layout (stacked, tabbed, default) |
155 |
bindsym Mod1+s layout stacking |
b/parser-specs/commands.spec
160 |
@@ -156,12 +156,28 @@ state KILL: |
161 |
end |
162 |
-> call cmd_kill($kill_mode) |
163 |
|
164 |
+# fullscreen enable|toggle [global] |
165 |
+# fullscreen disable |
166 |
# fullscreen [global] |
167 |
state FULLSCREEN: |
168 |
- fullscreen_mode = 'global' |
169 |
- -> call cmd_fullscreen($fullscreen_mode) |
170 |
+ action = 'disable' |
171 |
+ -> call cmd_fullscreen($action, "output") |
172 |
+ action = 'enable', 'toggle' |
173 |
+ -> FULLSCREEN_MODE |
174 |
+ action = '' |
175 |
+ -> FULLSCREEN_COMPAT |
176 |
+ |
177 |
+state FULLSCREEN_MODE: |
178 |
+ mode = 'global' |
179 |
+ -> call cmd_fullscreen($action, $mode) |
180 |
end |
181 |
- -> call cmd_fullscreen($fullscreen_mode) |
182 |
+ -> call cmd_fullscreen($action, "output") |
183 |
+ |
184 |
+state FULLSCREEN_COMPAT: |
185 |
+ mode = 'global' |
186 |
+ -> call cmd_fullscreen("toggle", $mode) |
187 |
+ end |
188 |
+ -> call cmd_fullscreen("toggle", "output") |
189 |
|
190 |
# split v|h|vertical|horizontal |
191 |
state SPLIT: |
b/src/commands.c
196 |
@@ -1574,20 +1574,27 @@ void cmd_focus(I3_CMD) { |
197 |
} |
198 |
|
199 |
/* |
200 |
- * Implementation of 'fullscreen [global]'. |
201 |
+ * Implementation of 'fullscreen (enabled|toggle [global])|disable' |
202 |
* |
203 |
*/ |
204 |
-void cmd_fullscreen(I3_CMD, char *fullscreen_mode) { |
205 |
- if (fullscreen_mode == NULL) |
206 |
- fullscreen_mode = "output"; |
207 |
- DLOG("toggling fullscreen, mode = %s\n", fullscreen_mode); |
208 |
+void cmd_fullscreen(I3_CMD, char *action, char *fullscreen_mode) { |
209 |
+ if (strcmp(action, "") == 0) |
210 |
+ action = "toggle"; |
211 |
+ fullscreen_mode_t mode = strcmp(fullscreen_mode, "global") == 0 ? CF_GLOBAL : CF_OUTPUT; |
212 |
+ DLOG("%s fullscreen, mode = %s\n", action, fullscreen_mode); |
213 |
owindow *current; |
214 |
|
215 |
HANDLE_EMPTY_MATCH; |
216 |
|
217 |
TAILQ_FOREACH(current, &owindows, owindows) { |
218 |
DLOG("matching: %p / %s\n", current->con, current->con->name); |
219 |
- con_toggle_fullscreen(current->con, (strcmp(fullscreen_mode, "global") == 0 ? CF_GLOBAL : CF_OUTPUT)); |
220 |
+ if (strcmp(action, "toggle") == 0) { |
221 |
+ con_toggle_fullscreen(current->con, mode); |
222 |
+ } else if (strcmp(action, "enable") == 0) { |
223 |
+ con_enable_fullscreen(current->con, mode); |
224 |
+ } else if (strcmp(action, "disable") == 0) { |
225 |
+ con_disable_fullscreen(current->con); |
226 |
+ } |
227 |
} |
228 |
|
229 |
cmd_output->needs_tree_render = true; |
b/src/con.c
234 |
@@ -565,37 +565,27 @@ void con_fix_percent(Con *con) { |
235 |
* |
236 |
*/ |
237 |
void con_toggle_fullscreen(Con *con, int fullscreen_mode) { |
238 |
- Con *workspace, *fullscreen; |
239 |
- |
240 |
if (con->type == CT_WORKSPACE) { |
241 |
DLOG("You cannot make a workspace fullscreen.\n"); |
242 |
return; |
243 |
} |
244 |
|
245 |
DLOG("toggling fullscreen for %p / %s\n", con, con->name); |
246 |
- if (con->fullscreen_mode == CF_NONE) { |
247 |
- /* 1: check if there already is a fullscreen con */ |
248 |
- if (fullscreen_mode == CF_GLOBAL) |
249 |
- fullscreen = con_get_fullscreen_con(croot, CF_GLOBAL); |
250 |
- else { |
251 |
- workspace = con_get_workspace(con); |
252 |
- fullscreen = con_get_fullscreen_con(workspace, CF_OUTPUT); |
253 |
- } |
254 |
- if (fullscreen != NULL) { |
255 |
- /* Disable fullscreen for the currently fullscreened |
256 |
- * container and enable it for the one the user wants |
257 |
- * to have in fullscreen mode. */ |
258 |
- LOG("Disabling fullscreen for (%p/%s) upon user request\n", |
259 |
- fullscreen, fullscreen->name); |
260 |
- fullscreen->fullscreen_mode = CF_NONE; |
261 |
- } |
262 |
|
263 |
- /* 2: enable fullscreen */ |
264 |
- con->fullscreen_mode = fullscreen_mode; |
265 |
- } else { |
266 |
- /* 1: disable fullscreen */ |
267 |
- con->fullscreen_mode = CF_NONE; |
268 |
- } |
269 |
+ if (con->fullscreen_mode == CF_NONE) |
270 |
+ con_enable_fullscreen(con, fullscreen_mode); |
271 |
+ else |
272 |
+ con_disable_fullscreen(con); |
273 |
+} |
274 |
+ |
275 |
+/* |
276 |
+ * Sets the specified fullscreen mode for the given container, sends the |
277 |
+ * “fullscreen_mode” event and changes the XCB fullscreen property of the |
278 |
+ * container’s window, if any. |
279 |
+ * |
280 |
+ */ |
281 |
+static void con_set_fullscreen_mode(Con *con, fullscreen_mode_t fullscreen_mode) { |
282 |
+ con->fullscreen_mode = fullscreen_mode; |
283 |
|
284 |
DLOG("mode now: %d\n", con->fullscreen_mode); |
285 |
|
286 |
@@ -619,6 +609,88 @@ void con_toggle_fullscreen(Con *con, int fullscreen_mode) { |
287 |
} |
288 |
|
289 |
/* |
290 |
+ * Enables fullscreen mode for the given container, if necessary. |
291 |
+ * |
292 |
+ * If the container’s mode is already CF_OUTPUT or CF_GLOBAL, the container is |
293 |
+ * kept fullscreen but its mode is set to CF_GLOBAL and CF_OUTPUT, |
294 |
+ * respectively. |
295 |
+ * |
296 |
+ * Other fullscreen containers will be disabled first, if they hide the new |
297 |
+ * one. |
298 |
+ * |
299 |
+ */ |
300 |
+void con_enable_fullscreen(Con *con, fullscreen_mode_t fullscreen_mode) { |
301 |
+ if (con->type == CT_WORKSPACE) { |
302 |
+ DLOG("You cannot make a workspace fullscreen.\n"); |
303 |
+ return; |
304 |
+ } |
305 |
+ |
306 |
+ assert(fullscreen_mode == CF_GLOBAL || fullscreen_mode == CF_OUTPUT); |
307 |
+ |
308 |
+ if (fullscreen_mode == CF_GLOBAL) |
309 |
+ DLOG("enabling global fullscreen for %p / %s\n", con, con->name); |
310 |
+ else |
311 |
+ DLOG("enabling fullscreen for %p / %s\n", con, con->name); |
312 |
+ |
313 |
+ if (con->fullscreen_mode == fullscreen_mode) { |
314 |
+ DLOG("fullscreen already enabled for %p / %s\n", con, con->name); |
315 |
+ return; |
316 |
+ } |
317 |
+ |
318 |
+ /* For both fullscreen modes, disable global fullscreen, if any. */ |
319 |
+ Con *fullscreen; |
320 |
+ if (fullscreen_mode == CF_GLOBAL || fullscreen_mode == CF_OUTPUT) { |
321 |
+ fullscreen = con_get_fullscreen_con(croot, CF_GLOBAL); |
322 |
+ if (fullscreen != NULL) |
323 |
+ con_disable_fullscreen(fullscreen); |
324 |
+ } |
325 |
+ /* For output fullscreen, disable other output fullscreen, if any. */ |
326 |
+ Con *workspace = con_get_workspace(con); |
327 |
+ if (fullscreen_mode == CF_OUTPUT) { |
328 |
+ fullscreen = con_get_fullscreen_con(workspace, CF_OUTPUT); |
329 |
+ if (fullscreen != NULL) |
330 |
+ con_disable_fullscreen(fullscreen); |
331 |
+ } |
332 |
+ |
333 |
+ /* Set focus to new fullscreen container. If on another workspace restore |
334 |
+ * focus afterwards. */ |
335 |
+ Con *old_focused = focused; |
336 |
+ con_focus(con); |
337 |
+ if (workspace != con_get_workspace(old_focused)) |
338 |
+ con_focus(old_focused); |
339 |
+ |
340 |
+ con_set_fullscreen_mode(con, fullscreen_mode); |
341 |
+} |
342 |
+ |
343 |
+/* |
344 |
+ * Disables fullscreen mode for the given container regardless of the mode, if |
345 |
+ * necessary. |
346 |
+ * |
347 |
+ */ |
348 |
+void con_disable_fullscreen(Con *con) { |
349 |
+ if (con->type == CT_WORKSPACE) { |
350 |
+ DLOG("You cannot make a workspace fullscreen.\n"); |
351 |
+ return; |
352 |
+ } |
353 |
+ |
354 |
+ DLOG("disabling fullscreen for %p / %s\n", con, con->name); |
355 |
+ |
356 |
+ if (con->fullscreen_mode == CF_NONE) { |
357 |
+ DLOG("fullscreen already disabled for %p / %s\n", con, con->name); |
358 |
+ return; |
359 |
+ } |
360 |
+ |
361 |
+ /* Restore focus to output-fullscreen container on workspace, when leaving |
362 |
+ * global-fullscreen. */ |
363 |
+ Con *workspace = con_get_workspace(con); |
364 |
+ Con *fullscreen = con_get_fullscreen_con(workspace, CF_OUTPUT); |
365 |
+ if (fullscreen != NULL) |
366 |
+ con_focus(con_descend_focused(fullscreen)); |
367 |
+ |
368 |
+ con_set_fullscreen_mode(con, CF_NONE); |
369 |
+} |
370 |
+ |
371 |
+/* |
372 |
* Moves the given container to the currently focused container on the given |
373 |
* workspace. |
374 |
* |
b/testcases/i3-test.config
379 |
@@ -20,7 +20,7 @@ bindsym Mod1+h split h |
380 |
bindsym Mod1+v split v |
381 |
|
382 |
# Fullscreen (Mod1+f) |
383 |
-bindsym Mod1+f fullscreen |
384 |
+bindsym Mod1+f fullscreen toggle |
385 |
|
386 |
# Stacking (Mod1+s) |
387 |
bindsym Mod1+s layout stacking |