Now you can select the level of aggressiveness when removing window borders.

slop.1

@@ -39,8 +39,8 @@ This sets the vertex shader, and fragment shader combo to use when drawing the f
 .BR \-f ", " \-\-format=\fISTRING\fR
 Sets the output format for slop. Format specifiers are %x, %y, %w, %h, %i, and %g. If actual percentage signs are desired in output, use a double percentage sign like so `%%`.
 .TP
-.BR \-n ", " \-\-nodecorations
-Attempt to select child windows, instead of the surface-level windows. This will typically avoid window decorations.
+.BR \-n ", " \-\-nodecorations=INT
+Sets the level of aggressiveness when trying to remove window decroations. `0' is off, `1' will try lightly to remove decorations, and `2' will recursively descend into the root tree until it gets the deepest available visible child under the mouse. Defaults to `0'.
 .TP
 .BR \-l ", " \-\-highlight
 Instead of outlining a selection, slop will highlight it instead. This is particularly useful if the color is set to an opacity lower than 1.

src/main.cpp

@@ -38,7 +38,7 @@ SlopOptions* getOptions( Options& options ) {
     foo->b = color.b;
     foo->a = color.a;
     options.getBool("highlight", 'l', foo->highlight);
-    options.getBool("nodecorations", 'n', foo->nodecorations);
+    options.getInt("nodecorations", 'n', foo->nodecorations);
     return foo;
 }
 
@@ -108,8 +108,12 @@ void printHelp() {
 	std::cout << "                                Set the selection rectangle's color. Supports\n";
 	std::cout << "                                  RGB or RGBA values.\n";
 	std::cout << "                                  (default=`0.5,0.5,0.5,1')\n";
-	std::cout << "  -n, --nodecorations           Attempt to select child windows in order to\n";
-	std::cout << "                                  avoid window decorations.  (default=off)\n";
+	std::cout << "  -n, --nodecorations=INT           Attempt to select child windows in order to\n";
+	std::cout << "                                  avoid window decorations. Setting this to\n";
+    std::cout << "                                  1 will enable a light attempt to\n";
+    std::cout << "                                  remove decorations. Setting this to 2 will\n";
+    std::cout << "                                  enable an aggressive decoration removal.\n";
+    std::cout << "                                  (default=`0')\n";
 	std::cout << "  -l, --highlight               Instead of outlining selections, slop\n";
 	std::cout << "                                  highlights it. This is only useful when\n";
 	std::cout << "                                  --color is set to a transparent color.\n";

src/mouse.cpp

@@ -39,7 +39,7 @@ void Mouse::setCursor( int cursor ) {
                               xcursor, CurrentTime );
 }
 
-Mouse::Mouse(X11* x11, bool nodecorations, Window ignoreWindow ) {
+Mouse::Mouse(X11* x11, int nodecorations, Window ignoreWindow ) {
     this->x11 = x11;
     currentCursor = XC_cross;
     xcursor = XCreateFontCursor( x11->display, XC_cross );
@@ -83,10 +83,10 @@ Window Mouse::findWindow( Window foo ) {
     unsigned int nchildren;
     Window selectedWindow;
     XQueryTree( x11->display, foo, &root, &parent, &children, &nchildren );
-    // The children are ordered, so we traverse backwards.
     if ( !children || nchildren <= 0 ) {
         return foo;
     }
+    // The children are ordered, so we traverse backwards.
     for( int i=nchildren-1;i>=0;i-- ) {
         if ( children[i] == ignoreWindow ) {
             continue;
@@ -103,12 +103,20 @@ Window Mouse::findWindow( Window foo ) {
         if ( a <= rect.z && a >= 0 ) {
             if ( b <= rect.w && b >= 0 ) {
                 selectedWindow = children[i];
-                if ( !nodecorations ) {
-                    XFree(children);
-                    return selectedWindow;
-                } else {
-                    XFree(children);
-                    return findWindow( selectedWindow );
+                switch( nodecorations ) {
+                    case 0:
+                        XFree(children);
+                        return selectedWindow;
+                    case 1:
+                        XFree(children);
+                        //return findWindow( selectedWindow );
+                        XQueryTree( x11->display, selectedWindow, &root, &parent, &children, &nchildren );
+                        if ( !children || nchildren <= 0 ) {
+                            return selectedWindow;
+                        }
+                        return children[nchildren-1];
+                    case 2:
+                        return findWindow( selectedWindow );
                 }
             }
         }

src/mouse.hpp

@@ -36,12 +36,12 @@ private:
     Cursor xcursor;
     int currentCursor;
     Window findWindow( Window foo );
-    bool nodecorations;
+    int nodecorations;
     Window ignoreWindow;
 public:
 	Window hoverWindow;
 	void update();
-    Mouse( X11* x11, bool nodecorations, Window ignoreWindow );
+    Mouse( X11* x11, int nodecorations, Window ignoreWindow );
     ~Mouse();
     void setCursor( int cursor );
     glm::vec2 getMousePos();

src/options.hpp

@@ -30,7 +30,7 @@
 
 static std::string validStringArguments[] = { "bordersize", "padding", "color", "shader", "highlight", "format", "tolerance", "nodecorations", "nokeyboard", "help", "xdisplay", "version" };
 static char validCharArguments[] = { 'b', 'p', 'c', 's', 'l', 'f', 't', 'n', 'k', 'h', 'x', 'v' };
-static unsigned int isFlagArgument[] = { false, false, false, false, true, false, false, true, true, true, false, true };
+static unsigned int isFlagArgument[] = { false, false, false, false, true, false, false, false, true, true, false, true };
 static unsigned int validArgumentCount = 12;
 static unsigned int maxFloatingValues = 0;
 

src/slop.hpp

@@ -44,7 +44,7 @@ public:
     float tolerance;
     bool highlight;
     bool nokeyboard;
-    bool nodecorations;
+    int nodecorations;
     std::string shader;
     float r;
     float g;