Re: [PATCH] rewrite of maximize functions

2013-04-17 Thread Carlos R. Mafra
On Thu, 18 Apr 2013 at 0:10:12 -0300, Renan Traba wrote: > simplified logic of handleMaximize function You should write a separate patch for this item alone. > added maximize on top and bottom half and added maximize at 4 corners: > left top, right top, left bottom, right bottom ditto. > remov

[PATCH] rewrite of maximize functions

2013-04-17 Thread Renan Traba
simplified logic of handleMaximize function added maximize on top and bottom half and added maximize at 4 corners: left top, right top, left bottom, right bottom removed unneeded old_maximize flags added to WPrefs options to configure new maximize modes --- WPrefs.app/KeyboardShortcuts.c | 12 ++

[repo.or.cz] wmaker-crm.git branch next updated: wmaker-0.95.4-89-g29a5267

2013-04-17 Thread crmafra
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project wmaker-crm.git. The branch, next has been updated via 29a526748553e11a1bcd7400e4cfc16c1f5351ed (commit) via 41da1b30dbfe09aef

Re: [PATCH 3/8] Add the application to the wapp_list

2013-04-17 Thread Rodolfo García Peñas (kix)
Quoting "Carlos R. Mafra" : On Sun, 14 Apr 2013 at 23:13:28 +0200, Rodolfo García Peñas (kix) wrote: From: "Rodolfo García Peñas (kix)" The screen has a list of applications: 8< kix@debian:~/src/wmaker/wmaker-crm/src$ grep wapp_list screen.h struct WAppli

Re: [PATCH 3/8] Add the application to the wapp_list

2013-04-17 Thread Carlos R. Mafra
On Sun, 14 Apr 2013 at 23:13:28 +0200, Rodolfo García Peñas (kix) wrote: > From: "Rodolfo García Peñas (kix)" > > The screen has a list of applications: > > 8< > kix@debian:~/src/wmaker/wmaker-crm/src$ grep wapp_list screen.h > struct WApplication *wapp_list;

Re: [PATCH 4/8] Split dock menu creation

2013-04-17 Thread Rodolfo García Peñas (kix)
Quoting "Carlos R. Mafra" : On Wed, 17 Apr 2013 at 6:37:57 +, Rodolfo García Peñas (kix) wrote: Hi Carlos, something is wrong in this patch. Please, don't apply it yet. I was not planning to apply it. I don't think that all that code movement is worth the trouble. We will need some p

Move the "Add drawer" option down

2013-04-17 Thread Rodolfo García Peñas (kix)
IMO, the option "Add drawer" should be moved down in the dock's menu. IMO, the right list sort should be: - Launch - Bring here - Hide - Settings - Dock position - Add drawer - Kill I use sometimes the "Launch" option and now I am adding drawers every time. kix Rodolfo García Peñas (kix) http

Re: [PATCH 6/8] Create clip only if needed

2013-04-17 Thread Carlos R. Mafra
On Wed, 17 Apr 2013 at 8:56:02 +, Rodolfo García Peñas (kix) wrote: > > >>When we set the with_clip flag is only for that function flow, but not > >>for all the wmaker session. > > > >I don't see why these things have to be different. If noclip is set in > >WPrefs the user already told wmaker

Re: [PATCH 6/8] Create clip only if needed

2013-04-17 Thread Rodolfo García Peñas (kix)
Quoting "Carlos R. Mafra" : On Tue, 16 Apr 2013 at 23:25:10 +0200, "Rodolfo García Peñas (kix)" wrote: On 16/04/2013 22:06, Carlos R. Mafra wrote: >> +int wWorkspaceNew(WScreen *scr, Bool with_clip) >> { >>WWorkspace *wspace, **list; >>int i; >> @@ -102,7 +102,7 @@ int wWorks

Re: [PATCH 4/8] Split dock menu creation

2013-04-17 Thread Carlos R. Mafra
On Wed, 17 Apr 2013 at 6:37:57 +, Rodolfo García Peñas (kix) wrote: > Hi Carlos, > > something is wrong in this patch. Please, don't apply it yet. I was not planning to apply it. I don't think that all that code movement is worth the trouble. -- To unsubscribe, send mail to wmaker-dev-uns

Re: [PATCH 6/8] Create clip only if needed

2013-04-17 Thread Carlos R. Mafra
On Tue, 16 Apr 2013 at 23:25:10 +0200, "Rodolfo García Peñas (kix)" wrote: > On 16/04/2013 22:06, Carlos R. Mafra wrote: > > >> +int wWorkspaceNew(WScreen *scr, Bool with_clip) > >> { > >>WWorkspace *wspace, **list; > >>int i; > >> @@ -102,7 +102,7 @@ int wWorkspaceNew(WScreen *scr) > >>