Introduce container_is_scratchpad_hidden

Just a convenience function that improves readability of the code.

Other things worth noting:

* container_get_siblings and container_sibling_index no longer use the
const keyword
* container_handle_fullscreen_reparent is only ever called after
attaching the container to a workspace, so its con->workspace check has
been changed to an assertion

1 files changed, 3 insertions, 3 deletions

diff --git a/sway/commands/sticky.c b/sway/commands/sticky.c
index e79af8af..5b70199c 100644
--- a/sway/commands/sticky.c
+++ b/sway/commands/sticky.c
@@ -17,15 +17,15 @@ struct cmd_results *cmd_sticky(int argc, char **argv) {
                 return error;
         }
         struct sway_container *container = config->handler_context.container;
-        
+
         if (container == NULL) {
                 return cmd_results_new(CMD_FAILURE, "No current container");
         };
-        
+
         container->is_sticky = parse_boolean(argv[0], container->is_sticky);
 
         if (container->is_sticky && container_is_floating_or_child(container) &&
-                        (!container->scratchpad || container->workspace)) {
+                        !container_is_scratchpad_hidden(container)) {
                 // move container to active workspace
                 struct sway_workspace *active_workspace =
                         output_get_active_workspace(container->workspace->output);