----- Rodolfo García Peñas (kix) <[email protected]> a écrit :
> The clip, dock, workspaces and drawers menus are common for all
> workspaces, so they should be included in the WWorkspace_info struct.
> 
> Signed-off-by: Rodolfo García Peñas (kix) <[email protected]>
> ---
>  [...]
> 
> diff --git a/src/workspace.h b/src/workspace.h
> index 0c434bd..56f7cfa 100644
> --- a/src/workspace.h
> +++ b/src/workspace.h
> @@ -37,6 +37,13 @@ typedef struct WWorkspace_info {
>  
>       /* Menus */
>       struct WMenu *workspace_menu;   /* workspace operation */
> +     struct WMenu *workspace_submenu; /* workspace list for window_menu */
> +     struct WMenu *clip_menu;        /* Menu for clips */
> +     struct WMenu *clip_submenu;     /* Workspace list for clips */
> +     struct WMenu *clip_options;     /* Options for Clip */
> +     struct WMenu *clip_ws_menu;     /* workspace menu for clip */
> +     struct WMenu *drawer_menu;      /* Menu for drawers */
> +     struct WMenu *dock_pos_menu;    /* Dock position menu */
>  } WWorkspace_info;


I have not checked how far this is linked to the workspace, but I would suggest 
instead:
  w_global.workspace.menu
  w_global.workspace.submenu
and then:
  w_global.clip.menu
  w_global.clip.submenu
  w_global.clip.options  (or 'opt_menu' to show that it is a menu)
  w_global.clip.ws_menu
and:
  w_global.drawer_menu  (or maybe 'w_global.drawer.menu' it there is intent in 
the future to add more global things for drawers)

  w_global.dock_pos_menu  (although a better name would be welcome, the current 
comment does not explain what this menu is all about)


--
To unsubscribe, send mail to [email protected].

Reply via email to